arpadboda commented on PR #9452: URL: https://github.com/apache/nifi/pull/9452#issuecomment-2441634223
@exceptionfactory The decoupling you mention would be beneficial I agree, but that's far from the scope of this PR and I don't feel like anyone in the community has the capacity to do that in a very short term. So in nutshell, here are the options I see: A) The restructuring you mention to completely separate iceberg nar from hive catalog would be something we all like, but it's a very long road and I don't see that happening in the near future B) Stay with the current, not too ideal structure, but update hive libs to v4, resolving the original goal of this discussion C) just remove Hive 3 (merging this PR as is), which in my opinion would cause more pain than gain. If Hive 3 would be by default built into NiFi, introducing vulnerabilities, I would be +1 here, but it's not the case, so there is no vulnerability here unless someone explicitly wants to build and run NiFi with these extensions. A is ideal but I see that very complex to happen in the near future, B seems feasible to me, in case of C I feel like it would introduce more pain than gain. -- 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]
