Louis Chu
Louis Chu
Thanks for picking it up @nung22 !
One more factor work: reduce the scope of using global context https://github.com/opensearch-project/dashboards-search-relevance/pull/267/files#r1312006788
There is some overlapping with https://github.com/opensearch-project/dashboards-search-relevance/issues/261
Related issue https://github.com/opensearch-project/dashboards-search-relevance/issues/184
> The version of druid used in legacy code path is outdated leading to certain issues with respect to security @jmashalk, could you elaborate on the specific security issues? We...
@jmashalk This CVE has no impact on SQL plugin because the concerning code block is a seperate path which loading static sources for druid web stats feature. Also verified in...
@dylan-tong-aws do you think this could go into sepecific plugins like ml-commons?
@cwperks The Flint Request Index is another system index. Will it be covered in this PR?
> @noCharger This index? https://github.com/opensearch-project/sql/blob/main/async-query-core/src/main/java/org/opensearch/sql/spark/execution/statestore/OpenSearchStateStoreUtil.java#L18 > > Should I add `.query_execution_request*` to the list of system index descriptors? Yes please, also could you point me if any additional change needed...
> > Yes please, also could you point me if any additional change needed for CI? > > @noCharger On the main branch, security is targeting Java 21. The CI...