In auto-discover, hide geocolumns from tables with multiple ones

[sharkAndshark]

Hi @nyurik I'm trying to resolve #627 .But it's a little bit hard battle against rust😂

[nyurik]

@sharkAndshark thanks for contributing! I think you can make your life much easier - just modify this code:

https://github.com/maplibre/martin/blob/34d16d28fcfc7add0cd6aaddf9b0a2d4df2a1710/src/pg/scripts/get_table_sources.sql

I would suggest creating a local db (e.g. run just start and just print-conn-str to see the DB conn string), connect to it using some postgres query tool, and modify the code to remove geometry columns from the query result.

Simplest -- just add

FILTER (WHERE columns.column_name IS NOT NULL AND columns.type_name != 'geometry')
instead of FILTER (WHERE columns.column_name IS NOT NULL)

[nyurik]

Thanks, looks good! Can you update the Readme to describe this change?

[sharkAndshark]

Yeah sure.

[nyurik]

Thanks, looks good!