jongyoul commented on PR #4783: URL: https://github.com/apache/zeppelin/pull/4783#issuecomment-2274620950
> I would like to see the actual benefits before pushing this change. for now, I only see breaking changes. We need to mitigate `zeppelin-interpreter` dependency. Currently, `zeppelin-interpreter` is quite big and has a lot of other dependencies unrelated to interpreters. To clean it up, dividing possible features with another module would be better. My goal is to merge zeppelin-interpreter, zeppelin-interpreter-parent and zeppelin-interpreter-shaded. :-) (I know the history but I believe that we can find a better way to manage them) Moreover, I agree that we change the name to `zeppelin-thrift` and so on as the first step. We also would better keep the package name to avoid breaking changes. we can do it later but we don't have to do it in the PR now. -- 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
