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

[GEOT-6050] Upgrade HSQLDB to 2.4.1 #1949

Merged
merged 1 commit into from Jun 27, 2018

Conversation

bencaradocdavies
Copy link
Member

@aaime
Copy link
Member

aaime commented Jun 26, 2018

I think it looks good, but I have a suggestion.

Let's say we have two GeoTools applications running off the gt-epsg-hsql database, one gets upgraded, the other does not. What happens? Is the old format preserved, or hsql updates the on disk format?
I believe it would be safer to bump the version number to 8.6.0.1, here:


This would give the newer version a different path, with no chance of conflict.

@bencaradocdavies
Copy link
Member Author

Sure, updated commit force-pushed. Can we also change the capitalisation of the path from /tmp/Geotools to /tmp/GeoTools. This has always bugged me. We still need the VERSION bump for the reasons you described above on case insensitive filesystems.

@bencaradocdavies
Copy link
Member Author

bencaradocdavies commented Jun 27, 2018

Oh my. I did it to myself. My Geotools to GeoTools change broke the test I added a few days ago. :-(

@bencaradocdavies
Copy link
Member Author

Once more, with feeling.

@aaime
Copy link
Member

aaime commented Jun 27, 2018

Yep, looking good and merging

@aaime aaime merged commit 9afcfc5 into geotools:master Jun 27, 2018
@bencaradocdavies bencaradocdavies deleted the geot-6050 branch June 27, 2018 20:47
@bencaradocdavies
Copy link
Member Author

Thanks @aaime.

@bencaradocdavies
Copy link
Member Author

If this change reduces the number of CoverageViewTest failures, it should be backported. I will run some test builds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants