ClassloaderInterpreter could also be removed.
Jeff Zhang <zjf...@gmail.com>于2017年5月10日周三 下午3:39写道: > > Hi moon > > Considering we use remote=true for interpreters now, and seems no user > complain about that. So I think we could remove it completely. Agree with > you that It could make code simpler, e.g. We don't need to check to use > RemoteAngularObjectRegistry or AngularObjectRegistry. > > > https://github.com/apache/zeppelin/blob/master/zeppelin-server/src/main/java/org/apache/zeppelin/socket/NotebookServer.java#L1458 > > > > Jeff Zhang <zjf...@gmail.com>于2017年5月8日周一 下午6:15写道: > >> Although it can bring some benefits, but there are 2 disadvantages for >> remote=false as I can think of >> >> 1. The interpreter log would mix with zeppelin server log. This may cause >> diagnosing difficult, markdown might be OK, but I am afraid it would be a >> problem for jdbc interpreter. >> 2. Extra memory consumption. This might be a problem for jdbc interpreter >> when it gets large amount of data from database. >> >> >> >> moon soo Lee <m...@apache.org>于2017年5月8日周一 下午5:27写道: >> >>> The option is legacy and being used in few unit tests as far as i >>> remember. >>> >>> I think we can either try completely remove this code (to keep code base >>> simple) or we can try re-introduce this feature (to optimize resource >>> usage, less restriction on data sharing between interpreters). >>> >>> I think interpreters like markdown, jdbc will have no big problem with >>> remote=false while Spark interpreter require separate JVM and remote=true >>> is required. >>> >>> remote=false will provide some chances to save some system resource >>> because >>> it does not run separate JVM process for interpreter. >>> >>> Also communication between interpreter (through ResourcePool) can be much >>> faster and have less restricted on object serialization. >>> >>> So remote=false option definitely provides some advantages. >>> However, keeping code base for remote=false and remote=true introduces >>> more >>> work. >>> >>> What do you think? >>> >>> Thanks, >>> moon >>> >>> On Mon, May 8, 2017 at 8:13 PM Jeff Zhang <zjf...@gmail.com> wrote: >>> >>> > I see all the interpreter setting set remote of interpreter option as >>> true, >>> > and didn't find any false case in interpreter-setting.json, does anyone >>> > know what is the use case of remote of interpreter option is false. Is >>> this >>> > a legacy mode that should be removed ? >>> > >>> > >>> > >>> https://github.com/apache/zeppelin/blob/master/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java#L183 >>> > >>> >>