RKuttruff opened a new pull request, #239: URL: https://github.com/apache/incubator-sdap-nexus/pull/239
This PR makes the necessary changes to the `doms.doms_data` table & results storage and retrieval code such that no data is lost for the `/cdmsresults` endpoint. **NOTE** The changes in this PR will not be fully compatible with an existing deployments `doms_data` table as created by a version of SDAP without these changes. This PR includes a script to make the necessary alterations to the table (`/tools/update-doms-data-schema/update.py`); running this will add the necessary columns and format the variable data as needed, then drop the (now unused) old variable data column. Alternatively, the `doms_data` (or even the whole `doms` keyspace) can be dropped if you don't mind losing all the old execution data. Without taking these steps, JSON rendered results will appear as they did prior to these changes and CSV and NetCDF rendering will fail. **Additional Changes** - Greatly improved speed of result data insertion - Ensured the `id` field of insitu observations returned from matchup are: a) non-null, b) non-empty, and c) unique for measurements at the same lat+lon+time but at differing depths. This prevents insitu matches from being incorrectly excluded when rendering NetCDF results. - Made the `output` parameter case-insensitive -- 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: dev-unsubscr...@sdap.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
