Conversation
…, and funding info.
…orkflow steps from GitHub workflows
- Introduced a new `Dockerfile` for containerized deployment. - Added dependencies for Mockito in tests and Git-Commit-Id plugin for tracking builds. - Configured Maven plugins for better CI/CD integration: Surefire (test setup), JaCoCo (coverage), and Spring Boot executable jar.
… Dockerfile and ImportService.
… `import.sh`, adjust README instructions for clarity
…ierarchyCache refactor - Introduced namespace and geometry adjustments in FlatBuffers schemas. - Added `S2Geometry` utility to handle JTS-to-S2Polygon conversions. - Refactored `HierarchyCache` with improved S2Cell-based grid indexing for hierarchy lookups. - Adjusted tolerances in `GeometrySimplificationService` for improved precision. - Updated Maven plugins and dependencies for modular builds.
…c in ImportService and HierarchyCache - Added caching optimizations for hierarchy layers to avoid redundant lookups. - Enhanced RocksDB configuration with tailored options for different use cases, improving performance and resource management. - Removed unused `createNodeCache` method and adjusted temp database handling.
- Switched to batch-based processing in `ImportService` using `BlockingQueue` for efficient memory use. - Updated `ImportServiceTest` to remove redundant fields and simplify setup logic. - Minor cleanup and adjustments to align with virtual thread usage.
…and detailed metrics
…mproved readability
…s in non-TTY mode
…vice batching logic, and update HierarchyCache validation - Deleted unused `getCellId` method and corresponding test for cleaner S2Helper implementation. - Adjusted `ImportService` to improve batching via spatial sorting and buffer queuing. - Updated `HierarchyCache` to handle empty level cache scenarios with improved logic. - Commented out unused Maven compiler arguments in pom.xml to clarify configuration.
…S2 neighbor retrieval logic - Deleted redundant XML configuration file for administrative boundaries. - Disabled CSRF protection in `SecurityConfig` where unnecessary. - Simplified neighbor retrieval in `S2Helper` by utilizing `getAllNeighbors` for better readability and efficiency.
…ng shard expansion
…EADME example - Updated the default `--data-dir` path in `import.sh` from `./data` to `./`. - Modified README example to align with the updated `--data-dir` usage. - Removed an unused comment in `S2Helper` for clarity.
…latform thread support
… clean up application properties - Removed `maxNodes` and `s2Level` fields from `PaikkaConfiguration` as they are no longer in use. - Updated `ImportServiceTest` to include the new `config` parameter in `ImportService` initialization. - Cleaned up `application.properties` by removing unused entries (`paikka.max-nodes` and `paikka.s2-level`) and leaving `paikka.admin.password` blank for improved security hygiene.
…iguration - Removed `paikka.admin.password` usage from `PaikkaApplication` and properties files. - Cleaned up `ImportServiceTest` by removing unused `s2Level` field. - Simplified `tearDown` method in tests by removing redundant exception declaration.
…pom.xml - Deleted redundant comment for Maven compiler arguments to streamline configuration. - Removed outdated JTS dependency source comment.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.