[GitHub] [zeppelin] akishichinibu commented on a diff in pull request #4568: [ZEPPELIN-5883] Upgrade influxDB client version to 2.3.0

2023-02-27 Thread via GitHub
akishichinibu commented on code in PR #4568: URL: https://github.com/apache/zeppelin/pull/4568#discussion_r1119639379 ## influxdb/pom.xml: ## @@ -35,8 +35,8 @@ influxdb UTF-8 -1.7.0 -3.13.1 +2.3.0 +4.7.2 Review Comment:

[GitHub] [zeppelin] akishichinibu commented on a diff in pull request #4568: [ZEPPELIN-5883] Upgrade influxDB client version to 2.3.0

2023-02-27 Thread via GitHub
akishichinibu commented on code in PR #4568: URL: https://github.com/apache/zeppelin/pull/4568#discussion_r1119639379 ## influxdb/pom.xml: ## @@ -35,8 +35,8 @@ influxdb UTF-8 -1.7.0 -3.13.1 +2.3.0 +4.7.2 Review Comment:

[GitHub] [zeppelin] akishichinibu commented on a diff in pull request #4568: [ZEPPELIN-5883] Upgrade influxDB client version to 2.3.0

2023-02-27 Thread via GitHub
akishichinibu commented on code in PR #4568: URL: https://github.com/apache/zeppelin/pull/4568#discussion_r1119639379 ## influxdb/pom.xml: ## @@ -35,8 +35,8 @@ influxdb UTF-8 -1.7.0 -3.13.1 +2.3.0 +4.7.2 Review Comment:

[GitHub] [zeppelin] akishichinibu commented on a diff in pull request #4568: [ZEPPELIN-5883] Upgrade influxDB client version to 2.3.0

2023-02-27 Thread via GitHub
akishichinibu commented on code in PR #4568: URL: https://github.com/apache/zeppelin/pull/4568#discussion_r1119639379 ## influxdb/pom.xml: ## @@ -35,8 +35,8 @@ influxdb UTF-8 -1.7.0 -3.13.1 +2.3.0 +4.7.2 Review Comment:

[GitHub] [zeppelin] akishichinibu commented on a diff in pull request #4568: [ZEPPELIN-5883] Upgrade influxDB client version to 2.3.0

2023-02-27 Thread via GitHub
akishichinibu commented on code in PR #4568: URL: https://github.com/apache/zeppelin/pull/4568#discussion_r1119639379 ## influxdb/pom.xml: ## @@ -35,8 +35,8 @@ influxdb UTF-8 -1.7.0 -3.13.1 +2.3.0 +4.7.2 Review Comment:

[GitHub] [zeppelin] phongnt commented on pull request #4570: [ZEPPELIN-5881] Missing dependency when package zeppelin-server module

2023-02-27 Thread via GitHub
phongnt commented on PR #4570: URL: https://github.com/apache/zeppelin/pull/4570#issuecomment-1447567707 @cometta I think Maven command in Dockerfile is not official for distribute docker image. I tried to start Zeppelin in docker by `docker run -p 8080:8080 --rm --name zeppelin

[GitHub] [zeppelin] cometta commented on pull request #4570: [ZEPPELIN-5881] Missing dependency when package zeppelin-server module

2023-02-27 Thread via GitHub
cometta commented on PR #4570: URL: https://github.com/apache/zeppelin/pull/4570#issuecomment-1446277819 i able to compile the master branch after your patching, but when i start python notebook example, run below cell ``` %python.ipython import sys sys? ``` i get

[GitHub] [zeppelin] Reamer commented on pull request #4570: [ZEPPELIN-5881] Missing dependency when package zeppelin-server module

2023-02-27 Thread via GitHub
Reamer commented on PR #4570: URL: https://github.com/apache/zeppelin/pull/4570#issuecomment-1446266520 If no further comments are received, I will merge the PullRequest on Wednesday. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to