Remove the dependency loop hack for useStatementTableController

[baurine]

Related PR: #749 (comment)

Do so for useSlowQueryTableContoller as well.

Code:

  // Notice: statements, tableColumns, selectedFields make loop dependencies
  const tableColumns = useMemo(
    () => statementColumns(statements, showFullSQL),
    [statements, showFullSQL]
  )
  // make selectedFields as a string instead of an array to avoid infinite loop
  // I have verified that it will cause infinite loop if we return selectedFields as an array
  // so it is better to use the basic type (string, number...) instead of object as the dependency
  const selectedFields = useMemo(
    () => getSelectedFields(visibleColumnKeys, tableColumns).join(','),
    [visibleColumnKeys, tableColumns]
  )

[unbyte]

is there any progress on solving this issue? If not, I'd like to create a new PR for it.

[baurine]

Not yet, please go ahead!