snazy commented on PR #785: URL: https://github.com/apache/polaris/pull/785#issuecomment-2597646700
> When this PR is ready for review, can you update the description or add an issue explaining _why_ this build change is necessary? This seems like it radically complicates the build to something that is much harder to understand and I think it would be good to clarify why that complexity is needed over the blackbox test setup that @dimas-b added previously. I've added a README to the PR hopefully explaining lots of it. The reason is to be able to use the plugin in the main Polaris code base - it _completely_ separates any kind of tests against the in-tree Polaris server. Another reason is that there is (naturally) no release of these plugins yet - so it would be impossible to use those. We can certainly later decide to move this stuff into a separate Git repo - but for now this is the actually only way to use the plugin(s). -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
