Re: VOTE thread for major SDAP change

2023-08-21 Thread Stepheny Perez
We received a +1 from the following individuals: - Riley Kuttruff - Nga Chung - Thomas Loubrieu - Thomas Huang I will go ahead and merge this PR (which has also been approved in GH). Thanks everyone for contributing and I look forward to getting this feature officially released! - Stepheny On

Re: VOTE thread for major SDAP change

2023-08-18 Thread Nga Chung
Thank you, Stepheny! The Jupyter notebook [1] that we've used to illustrate data match up was updated to be compatible with async job and pagination and works as expected. +1 (binding) from me. Best, Nga [1] https://github.com/access-cdms/cdms-notebooks/blob/master/CDMS-Match-Up-Demo.ipynb On

Re: [EXTERNAL] VOTE thread for major SDAP change

2023-08-16 Thread Loubrieu, Thomas G (US 398F)
] VOTE thread for major SDAP change +1 Yes Thanks for adding this major capability, Stepheny and team. It is good to see SDAP is supporting OGC Coverage standard. While SDAP is optimized for interactive analytics, thanks to Apache Spark, we know the response time is still depending on the volume

Re: [EXTERNAL] VOTE thread for major SDAP change

2023-08-16 Thread Huang, Thomas (US 398F)
+1 Yes Thanks for adding this major capability, Stepheny and team. It is good to see SDAP is supporting OGC Coverage standard. While SDAP is optimized for interactive analytics, thanks to Apache Spark, we know the response time is still depending on the volume of data involved in the computatio

Re: VOTE thread for major SDAP change

2023-08-16 Thread Riley Kuttruff
These changes have been extensively tested and improved through the Cloud-Based Data Matchup Service project's SDAP deployment and have been working like a charm. Queries that were previously infeasible due to their size & computation time are now able to be handled with ease. I will say that s

VOTE thread for major SDAP change

2023-08-15 Thread Stepheny Perez
Hi everyone, I opened a PR for a major change to SDAP here: https://github.com/apache/incubator-sdap-nexus/pull/249 Because this is a major change, I'll open a 72-hour VOTE thread and see if there are any concerns about this change. However you vote, please provide justification. This change