Thanks for bringing this up. I referenced the concerns that were mentioned on the PR about * the approach not using a `@Decorator`, mixing concerns. * exception/failure handling.
For me, these are important topics that need to be considered in the PR. It would be good to respect those concerns before we start reviewing this big-ish change, as there are possibly options to shrink it (immutables as an option?) and also consider exception/failure handling.
On 19.06.25 04:48, Adnan Hemani wrote:
Hi all, As per the comment on GitHub PR #1904 <https://github.com/apache/polaris/pull/1904>, I am opening this mailing thread to discuss the PR implementation. This PR is only adding instrumentation for additional events in `PolarisServiceImpl.java`. There is no additional business logic as part of this PR. There was one noted (and responded) remark about execution flow and one remark regarding cosmetic changes to the code, which is awaiting follow up by adutra@. I will let snazy@ take the floor from here as he requested this mail thread without being specific on his concerns on the PR comment. Best, Adnan Hemani
