RussellSpitzer commented on PR #2527: URL: https://github.com/apache/polaris/pull/2527#issuecomment-3293531077
> Re: Vert.x - it is currently used inside Quarkus. Yet, Quarkus is our standard runtime env. in this project, so I think it is fine for now. If there is a way (or need) to use Vert.x with this executor tool outside of Quarkus, let's do that in a separate follow-up PR. My question here is whether we need this PR at all, I'm not sure why we would want an abstraction layer over something which is a required component. I'm not sure having the flexibility here is worth the complexity. This PR is just a building block for something else so I think it's worth considering whether the PR is actually required or whether we can build everything without it. In this case I'm not sure why we wouldn't just have everything using Vert.X and not use an abstraction at all? -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
