[GitHub] yx91490 commented on a change in pull request #3287: IPythonInterpreter delete temp file and close stream

2019-01-23 Thread GitBox
yx91490 commented on a change in pull request #3287: IPythonInterpreter delete temp file and close stream URL: https://github.com/apache/zeppelin/pull/3287#discussion_r250476768 ## File path: python/src/main/java/org/apache/zeppelin/python/IPythonInterpreter.java ## @@

[GitHub] felixcheung commented on issue #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings

2019-01-23 Thread GitBox
felixcheung commented on issue #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings URL: https://github.com/apache/zeppelin/pull/3205#issuecomment-457059255 kicking off tests This is an automated message

[GitHub] felixcheung closed pull request #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings

2019-01-23 Thread GitBox
felixcheung closed pull request #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings URL: https://github.com/apache/zeppelin/pull/3205 This is an automated message from the Apache Git Service. To

[GitHub] egorklimov opened a new pull request #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings

2019-01-23 Thread GitBox
egorklimov opened a new pull request #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings URL: https://github.com/apache/zeppelin/pull/3205 ### What is this PR for? Now changes in table settings applies only after page refreshing which isn't convenient *

[GitHub] felixcheung commented on issue #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings

2019-01-23 Thread GitBox
felixcheung commented on issue #3205: [ZEPPELIN-3814] visualization - Add apply button to table settings URL: https://github.com/apache/zeppelin/pull/3205#issuecomment-457059125 I updated the PR title - hopeful someone might look at it. otherwise, merge if no more discussion

[GitHub] felixcheung merged pull request #3285: [ZEPPELIN-3957] Divide SparkIntegrationTest into parts for build under 50 minutes

2019-01-23 Thread GitBox
felixcheung merged pull request #3285: [ZEPPELIN-3957] Divide SparkIntegrationTest into parts for build under 50 minutes URL: https://github.com/apache/zeppelin/pull/3285 This is an automated message from the Apache Git

[GitHub] felixcheung commented on issue #3286: Fix abstractions. Removing Message class usage from NotebookService.

2019-01-23 Thread GitBox
felixcheung commented on issue #3286: Fix abstractions. Removing Message class usage from NotebookService. URL: https://github.com/apache/zeppelin/pull/3286#issuecomment-457056930 I think if it is "public" API, strongly typed param is better?

[GitHub] felixcheung commented on a change in pull request #3287: IPythonInterpreter delete temp file and close stream

2019-01-23 Thread GitBox
felixcheung commented on a change in pull request #3287: IPythonInterpreter delete temp file and close stream URL: https://github.com/apache/zeppelin/pull/3287#discussion_r250457908 ## File path: python/src/main/java/org/apache/zeppelin/python/IPythonInterpreter.java ##

[GitHub] fred521 commented on issue #3264: [ZEPPELIN-3908] Fix Date Sorting issue

2019-01-23 Thread GitBox
fred521 commented on issue #3264: [ZEPPELIN-3908] Fix Date Sorting issue URL: https://github.com/apache/zeppelin/pull/3264#issuecomment-457055731 the problem of the code is for loop in the line 166 can't sort for number, have to add extra check the variable is number or string

[GitHub] kennyishihara commented on issue #3264: [ZEPPELIN-3908] Fix Date Sorting issue

2019-01-23 Thread GitBox
kennyishihara commented on issue #3264: [ZEPPELIN-3908] Fix Date Sorting issue URL: https://github.com/apache/zeppelin/pull/3264#issuecomment-457051570 Any update to this? This is an automated message from the Apache Git

[GitHub] zjffdu commented on issue #3281: [ZEPPELIN-3943] - Add button "stop notebook execution"

2019-01-23 Thread GitBox
zjffdu commented on issue #3281: [ZEPPELIN-3943] - Add button "stop notebook execution" URL: https://github.com/apache/zeppelin/pull/3281#issuecomment-456794874 Thanks @Savalek LGTM This is an automated message from the

[GitHub] Savalek commented on a change in pull request #3281: [ZEPPELIN-3943] - Add button "stop notebook execution"

2019-01-23 Thread GitBox
Savalek commented on a change in pull request #3281: [ZEPPELIN-3943] - Add button "stop notebook execution" URL: https://github.com/apache/zeppelin/pull/3281#discussion_r250169708 ## File path: zeppelin-server/src/main/java/org/apache/zeppelin/service/NotebookService.java

Re: [VOTE] Release Apache Zeppelin 0.8.1 (RC1)

2019-01-23 Thread Jeff Zhang
Hi Felix, I am starting the release, it is moved to https://dist.apache.org/repos/dist/release/zeppelin/zeppelin-0.8.1/ Felix Cheung 于2019年1月23日周三 下午12:53写道: > Thanks Jeff. I just tried to reach the rc1 tarball and realized its gone ;) > > > -- > *From:* Jeff Zhang