[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-14 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2974 I think the green light can be easily mis-interpreted. If you have any other suggestion on the icon to use perhaps we could revisit. ---

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2974 hmm, I missed that statement in description. But I think it would be better to pop up a error message to users when it fails to restart. I don't think user would know to hover over that icon to

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-13 Thread bikassaha
Github user bikassaha commented on the issue: https://github.com/apache/zeppelin/pull/2974 The description says "on hovering the (circle) you will see success / error message for the respective interpreter." I wonder if that is going to show whats wrong in starting/restarting the

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-13 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2974 @sameer79 Or we could make it simple for now that the status icon means whether the interpreter restarting operation is successful just as what this PR did now IIUC. One concern is that when the

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-12 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2974 It looks like there's many misunderstandings between people about what the status icon really means. Personally I would suggest to abandon this PR for now, as I think end users would also has the

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-12 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2974 isn't it better to the loading/status light on the interpreter tab like in the screen shot in https://github.com/apache/zeppelin/pull/1155? ---

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-12 Thread sameer79
Github user sameer79 commented on the issue: https://github.com/apache/zeppelin/pull/2974 @r-kamath @prabhjyotsingh @bikassaha Awaiting for a review. ---

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-04 Thread sameer79
Github user sameer79 commented on the issue: https://github.com/apache/zeppelin/pull/2974 @bikassaha "Yes," The interpreter has successfully restarted. ---

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-04 Thread bikassaha
Github user bikassaha commented on the issue: https://github.com/apache/zeppelin/pull/2974 Does "ready to run" mean code can be executed in the note? ---

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-03 Thread sameer79
Github user sameer79 commented on the issue: https://github.com/apache/zeppelin/pull/2974 @bikassaha @felixcheung This is an extension to https://github.com/apache/zeppelin/pull/1155 The green icon does not mean the interpreter is running, it only indicates the

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-02 Thread bikassaha
Github user bikassaha commented on the issue: https://github.com/apache/zeppelin/pull/2974 Thats a good point. Though I thought interpreter restart was available from the note itself. Is that not the case? Agree interpreter controls/settings between admin/user needs to be looked at

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-02 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2974 hmm, but I can think of the opposite case too - that the interpreter is per-note and the user wants to have access to restart it, and not get locked down. sounds like a broader issue with

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-01 Thread bikassaha
Github user bikassaha commented on the issue: https://github.com/apache/zeppelin/pull/2974 I am guessing that would depend on whether we want to user to continue doing other stuff or make them wait for the interpreter to start/restart. The interpreter page is kind of a global

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-01 Thread sameer79
Github user sameer79 commented on the issue: https://github.com/apache/zeppelin/pull/2974 @bikassaha The current implementation is when the user confirms to restart of an interpreter the modal button text changes to "Saving Settings" with the spinner-icon and modal is there until

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-06-01 Thread bikassaha
Github user bikassaha commented on the issue: https://github.com/apache/zeppelin/pull/2974 Lets say I restarted the interpreter. Between that time and the interpreter actually starting, what icon is shown for the "start in progress state?" A blinking green? Also is this for

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-05-21 Thread sameer79
Github user sameer79 commented on the issue: https://github.com/apache/zeppelin/pull/2974 @Tagar Thanks for the comment, it will be handled with the new JIRA ---

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-05-21 Thread zjffdu
Github user zjffdu commented on the issue: https://github.com/apache/zeppelin/pull/2974 +1 for the idea or icon for interpreter process status ---

[GitHub] zeppelin issue #2974: ZEPPELIN-3472 No interpreter status is shown after res...

2018-05-21 Thread Tagar
Github user Tagar commented on the issue: https://github.com/apache/zeppelin/pull/2974 I think it's a great idea to add something like this to also show status if an interpreter is running. so show a green icon if interpreter is running/ or had been restarted successfully.