Andriy Redko

> Now that I remember the `ActionFilter` might not do the job since we wouldn't have access to the `ThreadPool/ThreadContext` in this apply method to assign labels for `QueryGrouping` use...

> @reta - The labeling logic will be based on the security context. For example - if the principal is Joe, the tenant is A. If the principal is Dave,...

I think the 1st option is pretty simple and straightforward, thanks @prudhvigodithi !

> This also looks to me like good target to split the gradle check in to multiple jobs and run the `MixedClusterClientYamlTestSuiteIT` separately and not with other tests @prudhvigodithi it...

> Hey @reta, what I was trying to say is not to run with ./gradlew check and run as a seperate job on a seperate runner. The intent is unclear...

> One easy way to mitigate this the linked pull requests can be a comma separated values rather than bullet point list (example [prudhvigodithi/opensearch-build#82](https://github.com/prudhvigodithi/opensearch-build/issues/82)). Hm ... not good ( Definitely...

> @reta Are you sure this applies here? As this is a client shouldn't it be in the same boat as the Java client and trying to preserve as much...

See please https://github.com/opensearch-project/performance-analyzer-rca/issues/591

@cwperks I have tons of questions regarding this change, it seems the scope of the change is to propagate the caller identity, is that right? If yes, we should: -...

Thanks @cwperks > This PR shows the use of the proxy pattern to show how it can keep track for RestHandler, but it doesn't cover other extension points. I think...