[GitHub] zeppelin issue #2444: [ZEPPELIN-2691] fix: should display setting for table

2018-02-08 Thread usmichael
Github user usmichael commented on the issue: https://github.com/apache/zeppelin/pull/2444 Hey @1ambda I don't mean to create overhead on this already closed PR but, I've 0.73v installed and this happens to me. Are there specific considerations I should have in mind to make it

[GitHub] zeppelin issue #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2444 Will merge this if no more discussion --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enab

[GitHub] zeppelin issue #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2444 @jongyoul Thanks for the review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled a

[GitHub] zeppelin issue #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/zeppelin/pull/2444 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fe

[GitHub] zeppelin issue #2444: [ZEPPELIN-2691] fix: should display setting for table

2017-06-27 Thread 1ambda
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2444 CI failed but irrelevant. - https://travis-ci.org/1ambda/zeppelin/jobs/247463188 ``` Results : Tests in error: ZeppelinIT.testSparkInterpreterDependencyLoa