dimas-b commented on PR #3396: URL: https://github.com/apache/polaris/pull/3396#issuecomment-3800447087
Re: indexing properties: > Is it necessitated by this PR? That's my only big question here, It's not obvious to me why the other changes in this PR caused this issue This PR integrates NoSQL code into the `runtime/server` module where Quarkus starts introspecting this code and issues the [warning](https://github.com/apache/polaris/pull/3396#discussion_r2727978172). Quarkus did not introspect this code until it was part of the server build. I'd say we ought to fix the warning from Quarkus in order to be sure that its introspection yields a complete picture for CDI to function properly. As I noted above, not having this property does not lead to any visible breakage, however, I think it is much safer to set it in order to benefit from Quarkus built-time checks to the full extent. -- 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]
