-
Notifications
You must be signed in to change notification settings - Fork 62
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
Optimize postgres tilestore #792
Conversation
c7ce774
to
13d3776
Compare
3f5dc06
to
cdb0e85
Compare
aaf81d1
to
9b469f4
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
/** | ||
* A record that holds the sql of a prepared statement and the number of parameters. | ||
* | ||
* @param sql |
Check notice
Code scanning / CodeQL
Spurious Javadoc @param tags Note
* A record that holds the sql of a prepared statement and the number of parameters. | ||
* | ||
* @param sql | ||
* @param parameters |
Check notice
Code scanning / CodeQL
Spurious Javadoc @param tags Note
This PR align the queries generated by the PostgresTileStore to the queries generated by the baremaps-exporter. There is still room to optimize the performance but it is a good first step. In addition, it greatly simplifies the code. @polastre Would you mind if I add you as a co-author for the squash commit? |
@bchapuis all good with me 👍 |
This is great! The queries are now much simpler and I suspect they will work out of the box with Apache Calcite as we don't use ST_Dump anymore. |
No description provided.