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

Post-processing SQL not running - pgosm-meta.sql #235

Closed
rustprooflabs opened this issue Mar 5, 2022 · 3 comments
Closed

Post-processing SQL not running - pgosm-meta.sql #235

rustprooflabs opened this issue Mar 5, 2022 · 3 comments
Labels
bug Something isn't working Docker Docker image and/or related script(s)
Milestone

Comments

@rustprooflabs
Copy link
Owner

What version of PgOSM Flex are you using?

v0.4.5

Docker image

Built locally from 0.4.5 tag.

What operating system, osm2pgsql, and PostgreSQL/PostGIS versions are you using?

N/A

What did you do exactly?

I was working on #226 and ran into a failure because the replication related procedures created in pgosm-meta.sql were not found. Upon investigation I verified that the output related to post-processing SQL was missing and the comments on tables, columns, etc. were not being added.

What did you expect to happen?

I expected the post-processing to work. I expect that if it fails for any reason the operator should be informed of the error. In the case the error is in the Docker image (like this issue) the make command should fail.

What did you do to try analyzing the problem?

Working on a fix. Adjusting the subprocess.run() call seems to be the trick. Should add a test to the Makefile to catch this error to prevent from happening again.

@rustprooflabs rustprooflabs added bug Something isn't working Docker Docker image and/or related script(s) labels Mar 5, 2022
@rustprooflabs rustprooflabs added this to the 0.4.6 milestone Mar 5, 2022
@rustprooflabs
Copy link
Owner Author

Limited to the pgosm-meta.sql script. Typo...

@rustprooflabs rustprooflabs changed the title Post-processing SQL not running Post-processing SQL not running - pgosm-meta.sql Mar 5, 2022
@rustprooflabs
Copy link
Owner Author

Commit acac426 fixes the error in the SQL script causing the issue.

Commits 6554d0e and e8f14a9 improves the post-processing to track errors by SQL script to report failures in post-processing. When this error is detected it is reported as having an error.

Post-processing pgosm-meta
Error in post-processing layerset: pgosm-meta
Post-processing amenity
Post-processing building

The final output flips to a warning to make the failure obvious and to point the user to the output above.

WARNING:pgosm-flex:pgosm_flex:PgOSM Flex completed with errors. Details in output

@rustprooflabs
Copy link
Owner Author

This is resolved in #226.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Docker Docker image and/or related script(s)
Projects
None yet
Development

No branches or pull requests

1 participant