Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Postgres imports: Hide postgis internal views #439

Merged
merged 1 commit into from
Jun 8, 2021

Conversation

craigds
Copy link
Member

@craigds craigds commented Jun 4, 2021

Description

Hides these views/tables from kart import '<postgres-url>':

 extname | extschema |      relname      | relkind
---------+-----------+-------------------+---------
 postgis | public    | spatial_ref_sys   | r
 postgis | public    | geography_columns | v
 postgis | public    | geometry_columns  | v
 postgis | public    | raster_columns    | v
 postgis | public    | raster_overviews  | v

These aren't importable things and they just cause potential problems,
especially when used with --all-tables (but they are noisy when
listing/prompting for tables also)

Related links:

re #434

Checklist:

  • Have you reviewed your own change?
  • Have you included test(s)?
  • Have you updated the changelog?

@craigds craigds requested a review from olsen232 June 4, 2021 00:49
@craigds craigds force-pushed the pg-hide-postgis-tables branch 2 times, most recently from 6ed53a9 to 42bf423 Compare June 4, 2021 02:10
Re #434

Hides these views/tables from `kart import '<postgres-url>'`:

```
 extname | extschema |      relname      | relkind
---------+-----------+-------------------+---------
 postgis | public    | spatial_ref_sys   | r
 postgis | public    | geography_columns | v
 postgis | public    | geometry_columns  | v
 postgis | public    | raster_columns    | v
 postgis | public    | raster_overviews  | v
```

These aren't importable things and they just cause potential problems,
especially when used with `--all-tables` (but they are noisy when
listing/prompting for tables also)
@craigds craigds merged commit 136b5fa into master Jun 8, 2021
@craigds craigds deleted the pg-hide-postgis-tables branch June 8, 2021 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants