[GitHub] [zeppelin] xiaomin0322 commented on pull request #4579: [ZEPPELIN-5893] NPE in ParagraphResult when getting progress from paragraphJson

2023-04-25 Thread via GitHub
xiaomin0322 commented on PR #4579: URL: https://github.com/apache/zeppelin/pull/4579#issuecomment-1522705230 > Could you please improve your description of the pull request. Have a look at #4568 for a good example. Adjusted -- This is an automated message from the Apache Git

[GitHub] [zeppelin] Reamer opened a new pull request, #4590: [ZEPPELIN-5903] Improve LRU NoteCache - Add a cleanup

2023-04-25 Thread via GitHub
Reamer opened a new pull request, #4590: URL: https://github.com/apache/zeppelin/pull/4590 ### What is this PR for? This pull request adds a cache cleanup method that attempts to remove an item from the cache that is not the eldest item and also does not have a read flag. ###

[jira] [Created] (ZEPPELIN-5903) NoteCache still growing

2023-04-25 Thread Philipp Dallig (Jira)
Philipp Dallig created ZEPPELIN-5903: Summary: NoteCache still growing Key: ZEPPELIN-5903 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5903 Project: Zeppelin Issue Type: Bug

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4582: [ZEPPELIN-5900] Fix RemoteSchedulerTest

2023-04-25 Thread via GitHub
Reamer commented on code in PR #4582: URL: https://github.com/apache/zeppelin/pull/4582#discussion_r1176399171 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java: ## @@ -342,17 +341,14 @@ public Scheduler getScheduler() { //

[GitHub] [zeppelin] zjffdu commented on pull request #4589: [ZEPPELIN-5898]fix:down csv data error

2023-04-25 Thread via GitHub
zjffdu commented on PR #4589: URL: https://github.com/apache/zeppelin/pull/4589#issuecomment-1521652049 Thanks for the contribution @zhugezifang , could you update the PR description? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [zeppelin] huage1994 commented on a diff in pull request #4582: [ZEPPELIN-5900] Fix RemoteSchedulerTest

2023-04-25 Thread via GitHub
huage1994 commented on code in PR #4582: URL: https://github.com/apache/zeppelin/pull/4582#discussion_r1176305796 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java: ## @@ -342,17 +341,14 @@ public Scheduler getScheduler() { //

[GitHub] [zeppelin] huage1994 commented on a diff in pull request #4582: [ZEPPELIN-5900] Fix RemoteSchedulerTest

2023-04-25 Thread via GitHub
huage1994 commented on code in PR #4582: URL: https://github.com/apache/zeppelin/pull/4582#discussion_r1176305796 ## zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/remote/RemoteInterpreter.java: ## @@ -342,17 +341,14 @@ public Scheduler getScheduler() { //

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

2023-04-25 Thread via GitHub
Reamer commented on PR #4568: URL: https://github.com/apache/zeppelin/pull/4568#issuecomment-1521465572 ping @akishichinibu Any JUnit tests to add? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [zeppelin] Reamer commented on pull request #4579: [ZEPPELIN-5893] NPE in ParagraphResult when getting progress from paragraphJson

2023-04-25 Thread via GitHub
Reamer commented on PR #4579: URL: https://github.com/apache/zeppelin/pull/4579#issuecomment-1521464643 Could you please improve your description of the pull request. Have a look at #4568 for a good example. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [zeppelin] xiaomin0322 commented on pull request #4579: [ZEPPELIN-5893] NPE in ParagraphResult when getting progress from paragraphJson

2023-04-25 Thread via GitHub
xiaomin0322 commented on PR #4579: URL: https://github.com/apache/zeppelin/pull/4579#issuecomment-1521444700 > @xiaomin0322 Could you update the jira title to be more specific? `Dev zeppelin client bug` is too vague. Something like `NPE in ParagraphResult when getting progress from

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4589: [ZEPPELIN-5898]fix:down csv data error

2023-04-25 Thread via GitHub
Reamer commented on code in PR #4589: URL: https://github.com/apache/zeppelin/pull/4589#discussion_r1176204154 ## zeppelin-web/src/app/notebook/paragraph/result/result.controller.js: ## @@ -895,6 +895,9 @@ function ResultCtrl($scope, $rootScope, $route, $window, $routeParams,

[GitHub] [zeppelin] Reamer commented on a diff in pull request #4586: [ZEPPELIN-5902] Support Spark 3.4

2023-04-25 Thread via GitHub
Reamer commented on code in PR #4586: URL: https://github.com/apache/zeppelin/pull/4586#discussion_r1176200080 ## spark/interpreter/pom.xml: ## @@ -518,6 +518,20 @@ + + spark-3.4 + +true Review Comment: Only one Spark Profile should have

[GitHub] [zeppelin] Reamer commented on pull request #4582: [ZEPPELIN-5900] Fix RemoteSchedulerTest

2023-04-25 Thread via GitHub
Reamer commented on PR #4582: URL: https://github.com/apache/zeppelin/pull/4582#issuecomment-1521223455 > BTW it's weird that the `RemoteSchedulerTest` used to runs fine on Junit 4, Do you know why? Unfortunately no. However, since only the timing matters due to the six threads

[GitHub] [zeppelin] Reamer commented on pull request #4581: [ZEPPELIN-5901] k8s interpreter pod launch fails

2023-04-25 Thread via GitHub
Reamer commented on PR #4581: URL: https://github.com/apache/zeppelin/pull/4581#issuecomment-1521210881 > when does it get merged ? When all tests have been successfully completed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on