| Re: [geogig-dev] SQLite backend for GeoGig |
Have a good weekend and I hope your daughter's performance and her upcoming summer were/are wonderful!I think that's enough for one PR. I don't want to mix too many changes in at once.Also, what I'm going to have in this first PR is a few Maven profiles to allow us to enable the "large tests" that are currently annotated with @Ignore. The profiles set a system property when Maven runs that the test can check. I use JUnit's Assume.assumeTrue(boolean) in those specific tests that way the tests are marked "skipped" if the profile is not active, and the tests are executed if the profile is active.This has, however, led me to the next hurdle with SQLite, which is the multithreading issues. The tests pass if I restrict Maven to a single process. But when I let Maven run the tests in parallel, they start to fail. But I think this was expected.OK, I think I have the test issues fixed (running them again to make sure).I had a couple other issues with the XerialConflictsDatabaseV2 impl (probably in XerialConflictsDatabase but I haven't checked yet). Both the "get" and "remove" methods were building SQL strings assuming namesapce and path were never null. I changed the SQL string being built there to omit the "WHERE" clauses if the namespace and/or path values passed in are null. I'll push it up once I make sure it isn't gonna break anything else and get a PR over to you.
