jongyoul commented on PR #4783: URL: https://github.com/apache/zeppelin/pull/4783#issuecomment-2308657171
> To address the above issue, I think we should separate the optional dependencies into individual modules, then we can eliminate them in some interpreters, but is zeppelin-protocol that case? I think it is required by all interpreters. Yes, you're right. It should be mandatory for all interpreters for now. However, it was because we only provide all functions within the zeppelin-interpreter module. If we provided them as a zeppelin-protocol including definition, I believe that we might have different python interpreter implementation like pure python module. :-) I want to give some chance to utilize thrift definition as contributors want to use. It's the first step. I don't know the future yet but I believe that it gives more choice to the community. > I fully agree with that, and I also noticed the packaging time for zeppelin-interpreter-shaded is pretty slow due to a lot of classes needing to be relocated, and some of those deps come from [ZEPPELIN-3471](https://issues.apache.org/jira/browse/ZEPPELIN-3471), I have raised a [thread](https://lists.apache.org/thread/vm2m4xg6bg0db33txrrm9ydd15dqy1lx) to remove it in the dev mailing list, please leave your comments to help the community make the decision. For the raft issue, let's move to the next step 👍 -- 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: reviews-unsubscr...@zeppelin.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
