ishnagy commented on PR #6146: URL: https://github.com/apache/hive/pull/6146#issuecomment-3502504681
> In any case, your issue is with hive-exec bringing transitive dependencies, isn't it? indeed. so marking the registry dep as provided/optional appears to be a plausible solution. \*to be confirmed, my builds are still running. **if the builds produce a positive result, I'd be happy to merge the dep changes only, without the code refactor.** OTOH, pulling in an extra hadoop dependency _just for grabbing a single String instance_ (a class name) feels like definite overkill. :) I understand that there might be other components relying on the hadoop registry functionality, but it would feel more comfortable identifying those (if there are any besides `hive-service`), and move the explicit dependency to the place(s) where it belongs. I'd be happy to do some extra investigation if you provide me with some pointers what should we check. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
