bchapuis commented on issue #849: URL: https://github.com/apache/incubator-baremaps/issues/849#issuecomment-2139180343
@sebr72 @Drabble I will merge the current PR and organize the git history to have three separated commits with our individual contributions. For the following tasks, I suggest we make individual PRs and split the work more clearly. - [ ] Cleanup the sonar problems when it makes sense (@sebr72 ). - [ ] Add some unit tests to the GeoParquetReader (@sebr72 ). - [ ] Define a better organization for the packages and classes of the geoparquet module (data, hadoop, etc.). - [ ] Improve the allocation of objects in the GeoParquet reader (list for each field, wrapper for each value, etc.) - [ ] The high level abstration (DataTable, DataColumn, etc.) needs support for nested data structures such as groups in geoparquet and json in other data formats (@bchapuis will make a first refactoring pass). - [ ] Improve the abstraction of the GeoParquetGroup (the current version uses internal classes, and the getters/setters were quickly definined to have an end-to-end example, etc.) - after doing a pass on the DataTable abstraction -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
