[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-24 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2802 rename the property in #2813 ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-23 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2802 Thanks @ejona86 ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-23 Thread ejona86
Github user ejona86 commented on the issue: https://github.com/apache/zeppelin/pull/2802 I just changed them to be more clear in grpc/grpc-java#4106. Each message gets its own gRPC frame, but the very fact there are gRPC frames is more of an internal detail; users shouldn't need to

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2802 @ejona86 The error from grpc is about `Frame size exceed` ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-21 Thread ejona86
Github user ejona86 commented on the issue: https://github.com/apache/zeppelin/pull/2802 @Tagar, I mis-read the error earlier. That message is just "gRPC message too large." It'd probably be good to talk about "messages" instead of "frames" here. ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-20 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2802 @zjffdu I just verified this new commit and it works great. Glad to see exceptions in ipython backend are now being captured correctly. I wasn't able to actually test frame size

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-20 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2802 LGTM thanks a lot @zjffdu ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-20 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2802 thank you @zjffdu - I will check this today. ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-19 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2802 @Tagar Have you tried this PR ? Does this PR solve your issue ? ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-18 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2802 > ERROR [2018-02-14 10:39:10,922] ({grpc-default-executor-3} IPythonClient.java[onError]:138) - Fail to call IPython grpc > io.grpc.StatusRuntimeException: RESOURCE_EXHAUSTED:

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-16 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2802 @Tagar Can you paste the full log ? I'd like to see the full stacktrace. ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-15 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2802 @zjffdu I don't have a reproducible case yet. This error hasn't been very consistent for me so it might be also a side effect of another problem. That's why I asked above if we could capture

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-15 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2802 @Tagar Do you have any sample code to reproduce this issue ? So that I can verify it and also may include it in unit test. ---

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-14 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2802 1) My main point was that this exception should be thrown to the user, so he or she has a chance to increase this limit. Currently if it breaks, only way to find out about this limitation is to

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-14 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2802 @Tagar , 4 Mb is the default value of gprc, I am fine to use another large. But from your error logs, your code will produce 200M data, I am curious to know how do you produce that, do you mind to

[GitHub] zeppelin issue #2802: ZEPPELIN-3236. Make grpc framesize configurable

2018-02-14 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2802 Thanks for the heads up, yep I figured out that I have to tune up zeppelin.ipython.grpc.framesize to a large number. I looked over the PR. Two quick suggestions 1) Would it be possible