[GitHub] jongyoul commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types

2018-12-20 Thread GitBox
jongyoul commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types URL: https://github.com/apache/zeppelin/pull/3244#issuecomment-449284850 @felixcheung Thanks a lot đź‘Ť This is an automated mes

[GitHub] felixcheung commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types

2018-12-20 Thread GitBox
felixcheung commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types URL: https://github.com/apache/zeppelin/pull/3244#issuecomment-449284008 @conker84 sorry, could you see if you could re open this PR so I can merge again?

[GitHub] felixcheung opened a new pull request #3270: Revert "[ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types"

2018-12-20 Thread GitBox
felixcheung opened a new pull request #3270: Revert "[ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types" URL: https://github.com/apache/zeppelin/pull/3270 Reverts apache/zeppelin#3244 This is an automated message

[GitHub] felixcheung closed pull request #3270: Revert "[ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types"

2018-12-20 Thread GitBox
felixcheung closed pull request #3270: Revert "[ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types" URL: https://github.com/apache/zeppelin/pull/3270 This is an automated message from the Apache Git Service. To r

[GitHub] felixcheung commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types

2018-12-20 Thread GitBox
felixcheung commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types URL: https://github.com/apache/zeppelin/pull/3244#issuecomment-449281875 Squash and merge seems like the right option, https://help.github.com/articles/merging-a-pull-request/

[GitHub] felixcheung removed a comment on issue #3267: [WIP][ZEPPELIN-3609] Refactoring `ZeppelinServer` to adopt DI

2018-12-20 Thread GitBox
felixcheung removed a comment on issue #3267: [WIP][ZEPPELIN-3609] Refactoring `ZeppelinServer` to adopt DI URL: https://github.com/apache/zeppelin/pull/3267#issuecomment-449281816 Squash and merge seems like the right option, https://help.github.com/articles/merging-a-pull-request/ --

[GitHub] felixcheung commented on issue #3267: [WIP][ZEPPELIN-3609] Refactoring `ZeppelinServer` to adopt DI

2018-12-20 Thread GitBox
felixcheung commented on issue #3267: [WIP][ZEPPELIN-3609] Refactoring `ZeppelinServer` to adopt DI URL: https://github.com/apache/zeppelin/pull/3267#issuecomment-449281816 Squash and merge seems like the right option, https://help.github.com/articles/merging-a-pull-request/ --

[GitHub] xueyumusic commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0

2018-12-20 Thread GitBox
xueyumusic commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0 URL: https://github.com/apache/zeppelin/pull/3266#issuecomment-449273098 I think it would be better to wait for flink-1.7.1 release, it seems to fix several bugs of 1.7.0. Thank you~ @zjffdu --

[GitHub] zjffdu commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0

2018-12-20 Thread GitBox
zjffdu commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0 URL: https://github.com/apache/zeppelin/pull/3266#issuecomment-449261087 @xueyumusic Flink community is starting the release of 1.7.1, do you mind to wait for the release of 1.7.1 ?

[GitHub] jongyoul commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types

2018-12-20 Thread GitBox
jongyoul commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types URL: https://github.com/apache/zeppelin/pull/3244#issuecomment-449073264 I've tested my wip pr by clicking squash and merge button on the pr page. I saw we could edit a title and commit messa

[GitHub] felixcheung commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types

2018-12-20 Thread GitBox
felixcheung commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types URL: https://github.com/apache/zeppelin/pull/3244#issuecomment-449071812 Github doesn’t let me edit title. I think committer should use the merge script and not the github Web UI. I

[GitHub] zjffdu commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0

2018-12-20 Thread GitBox
zjffdu commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0 URL: https://github.com/apache/zeppelin/pull/3266#issuecomment-448999111 @xueyumusic I did some work recently to enable table api for flink. Will create a new PR when it's ready. --

[GitHub] xueyumusic commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0

2018-12-20 Thread GitBox
xueyumusic commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0 URL: https://github.com/apache/zeppelin/pull/3266#issuecomment-448984312 Hi, @jongyoul , I enabled CI, the latest commit travis test is https://travis-ci.org/xueyumusic/zeppelin/builds/470364971, thanks ---

[jira] [Created] (ZEPPELIN-3919) Paragraph config default value can be customized

2018-12-20 Thread Xun Liu (JIRA)
Xun Liu created ZEPPELIN-3919: - Summary: Paragraph config default value can be customized Key: ZEPPELIN-3919 URL: https://issues.apache.org/jira/browse/ZEPPELIN-3919 Project: Zeppelin Issue Type:

[GitHub] jongyoul commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0

2018-12-20 Thread GitBox
jongyoul commented on issue #3266: [ZEPPELIN-3914] upgrade Flink to 1.7.0 URL: https://github.com/apache/zeppelin/pull/3266#issuecomment-448944022 BTW, could you please enable CI? Please check https://travis-ci.org/xueyumusic ---

[GitHub] jongyoul commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types

2018-12-20 Thread GitBox
jongyoul commented on issue #3244: [ZEPPELIN-3882] Neo4jInterpreter - Support Point and Date Types URL: https://github.com/apache/zeppelin/pull/3244#issuecomment-448928694 @felixcheung I think it would be better to squash and merge with some edit of title indicating an issue number. WDYT?

[GitHub] egorklimov commented on issue #3254: [ZEPPELIN-3575] Add 'Copy Column Name' to table visualisation-table

2018-12-20 Thread GitBox
egorklimov commented on issue #3254: [ZEPPELIN-3575] Add 'Copy Column Name' to table visualisation-table URL: https://github.com/apache/zeppelin/pull/3254#issuecomment-448921020 It seems to be important. What alternative solution did you find for table vizualization? --