[jira] [Created] (ZEPPELIN-5860) Update maven-checkstyle-plugin to 3.2.0

2022-12-02 Thread Jongyoul Lee (Jira)
Jongyoul Lee created ZEPPELIN-5860:
--

 Summary: Update maven-checkstyle-plugin to 3.2.0
 Key: ZEPPELIN-5860
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5860
 Project: Zeppelin
  Issue Type: Improvement
  Components: CI-infra
Reporter: Jongyoul Lee
Assignee: Jongyoul Lee


The current version of maven-checkstyle-plugin is 2.17 which is from 2015. I 
would like to add a new feature including disallowing multiple empty lines so 
I'll update it. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4525: Bump decode-uri-component from 0.2.0 to 0.2.2 in /zeppelin-web

2022-12-02 Thread GitBox


dependabot[bot] opened a new pull request, #4525:
URL: https://github.com/apache/zeppelin/pull/4525

   Bumps 
[decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) 
from 0.2.0 to 0.2.2.
   
   Release notes
   Sourced from https://github.com/SamVerschueren/decode-uri-component/releases";>decode-uri-component's
 releases.
   
   v0.2.2
   
   Prevent overwriting previously decoded tokens  980e0bf
   
   https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.1...v0.2.2";>https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.1...v0.2.2
   v0.2.1
   
   Switch to GitHub workflows  76abc93
   Fix issue where decode throws - fixes https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/6";>#6
  746ca5d
   Update license (https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/1";>#1)
  486d7e2
   Tidelift tasks  a650457
   Meta tweaks  66e1c28
   
   https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.0...v0.2.1";>https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.0...v0.2.1
   
   
   
   Commits
   
   https://github.com/SamVerschueren/decode-uri-component/commit/a0eea469d26eb0df668b081672cdb9581feb78eb";>a0eea46
 0.2.2
   https://github.com/SamVerschueren/decode-uri-component/commit/980e0bf09b64d94f1aa79012f895816c30ffd152";>980e0bf
 Prevent overwriting previously decoded tokens
   https://github.com/SamVerschueren/decode-uri-component/commit/3c8a373dd4837e89b3f970e01295dd03e1405a33";>3c8a373
 0.2.1
   https://github.com/SamVerschueren/decode-uri-component/commit/76abc939783fe3900fadb7d384a74d324d5557f3";>76abc93
 Switch to GitHub workflows
   https://github.com/SamVerschueren/decode-uri-component/commit/746ca5dcb6667c5d364e782d53c542830e4c10b9";>746ca5d
 Fix issue where decode throws - fixes https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/6";>#6
   https://github.com/SamVerschueren/decode-uri-component/commit/486d7e26d3a8c0fbe860fb651fe1bc98c2f2be30";>486d7e2
 Update license (https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/1";>#1)
   https://github.com/SamVerschueren/decode-uri-component/commit/a65045724e6234acef87f31da499d4807b20b134";>a650457
 Tidelift tasks
   https://github.com/SamVerschueren/decode-uri-component/commit/66e1c2834c0e189201cb65196ec3101372459b02";>66e1c28
 Meta tweaks
   See full diff in https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.0...v0.2.2";>compare
 view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=decode-uri-component&package-manager=npm_and_yarn&previous-version=0.2.0&new-version=0.2.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   - `@dependabot use these labels` will set the current labels as the default 
for future PRs for this repo and language
   - `@dependabot use these reviewers` will set the current reviewers as the 
default for future PRs for this repo and language
   - `@dependabot use these assignees` will set the current assignees as the 
default for future PRs for this repo and language
   - `@dependabot use this milestone` will set the current milestone as the 
default for future PRs for this repo and language
   
   You can disable automated security fix PRs for this re

[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4210: [ZEPPELIN-5501] Interpreter shade

2022-12-02 Thread GitBox


jongyoul commented on code in PR #4210:
URL: https://github.com/apache/zeppelin/pull/4210#discussion_r1038188188


##
zeppelin-plugins/notebookrepo/gcs/pom.xml:
##
@@ -35,11 +35,22 @@
 
 
 1.14.0
+
+20.0
 0.32.0-alpha
 0.27
 NotebookRepo/GCSNotebookRepo
 
 
+
+
+
+com.google.guava
+guava
+20.0

Review Comment:
   ```suggestion
   ${guava.version}
   ```



-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul closed pull request #4520: [MINOR] Fix Appveyor error

2022-12-02 Thread GitBox


jongyoul closed pull request #4520: [MINOR] Fix Appveyor error
URL: https://github.com/apache/zeppelin/pull/4520


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (ZEPPELIN-5861) Broken CI, because of faulty shading

2022-12-02 Thread Philipp Dallig (Jira)
Philipp Dallig created ZEPPELIN-5861:


 Summary: Broken CI, because of faulty shading
 Key: ZEPPELIN-5861
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5861
 Project: Zeppelin
  Issue Type: Bug
Reporter: Philipp Dallig
Assignee: Philipp Dallig


I think that the following error breaks our CI. Why this occurs now I do not 
know.

 
{code:java}
2022-12-02T13:48:14.4345264Z Interpreter download command: 
/opt/hostedtoolcache/Java_Temurin-Hotspot_jdk/8.0.352-8/x64/bin/java 
-Dfile.encoding=UTF-8 
-Dlog4j.configuration=file:///home/runner/work/zeppelin/zeppelin/conf/log4j.properties
 
-Dlog4j.configurationFile=file:///home/runner/work/zeppelin/zeppelin/conf/log4j2.properties
 
-Dzeppelin.log.file=/home/runner/work/zeppelin/zeppelin/logs/zeppelin-interpreter-test-shared_process--fv-az311-404.log
 -cp 
:/home/runner/work/zeppelin/zeppelin/zeppelin-zengine/../interpreter_TimeoutLifecycleManagerTest/test/*:/home/runner/work/zeppelin/zeppelin/zeppelin-interpreter-shaded/target/*:/home/runner/work/zeppelin/zeppelin/zeppelin-zengine/target/test-classes/*:::/home/runner/work/zeppelin/zeppelin/interpreter/zeppelin-interpreter-shaded-0.11.0-SNAPSHOT.jar:/home/runner/work/zeppelin/zeppelin/zeppelin-interpreter/target/classes:/home/runner/work/zeppelin/zeppelin/zeppelin-zengine/target/test-classes
 org.apache.zeppelin.interpreter.remote.RemoteInterpreterDownloader 127.0.0.1 
42175 test 
/home/runner/work/zeppelin/zeppelin/zeppelin-zengine/../local-repo/test
2022-12-02T13:48:14.4349276Z ##[error]Exception in thread "main" 
java.lang.IllegalArgumentException: Unable to create EvictionPolicy instance of 
type 
org/apache/zeppelin/shaded/org.apache.zeppelin.shaded.org.apache.commons.pool2.impl.DefaultEvictionPolicy
2022-12-02T13:48:14.4351024Zat 
org.apache.zeppelin.shaded.org.apache.commons.pool2.impl.BaseGenericObjectPool.setEvictionPolicyClassName(BaseGenericObjectPool.java:607)
2022-12-02T13:48:14.4352170Zat 
org.apache.zeppelin.shaded.org.apache.commons.pool2.impl.GenericObjectPool.setConfig(GenericObjectPool.java:321)
2022-12-02T13:48:14.4353132Zat 
org.apache.zeppelin.shaded.org.apache.commons.pool2.impl.GenericObjectPool.(GenericObjectPool.java:117)
2022-12-02T13:48:14.4353963Zat 
org.apache.zeppelin.interpreter.remote.PooledRemoteClient.(PooledRemoteClient.java:47)
2022-12-02T13:48:14.4354810Zat 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterEventClient.(RemoteInterpreterEventClient.java:67)
2022-12-02T13:48:14.4355743Zat 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterDownloader.main(RemoteInterpreterDownloader.java:58)
2022-12-02T13:48:14.4356736Z Caused by: java.lang.ClassNotFoundException: 
org/apache/zeppelin/shaded/org.apache.zeppelin.shaded.org.apache.commons.pool2.impl.DefaultEvictionPolicy
2022-12-02T13:48:14.4357471Zat java.lang.Class.forName0(Native Method)
2022-12-02T13:48:14.4357902Zat java.lang.Class.forName(Class.java:348)
2022-12-02T13:48:14.4358766Zat 
org.apache.zeppelin.shaded.org.apache.commons.pool2.impl.BaseGenericObjectPool.setEvictionPolicyClassName(BaseGenericObjectPool.java:598)
2022-12-02T13:48:14.4360161Z... 5 more {code}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Created] (ZEPPELIN-5862) Allow using the docker socket to start dockerized interpreter processes

2022-12-02 Thread Arnout Engelen (Jira)
Arnout Engelen created ZEPPELIN-5862:


 Summary: Allow using the docker socket to start dockerized 
interpreter processes
 Key: ZEPPELIN-5862
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5862
 Project: Zeppelin
  Issue Type: Improvement
  Components: docker
Reporter: Arnout Engelen


Currently, in the documentation for running the interpreters in Docker at 
[https://zeppelin.apache.org/docs/latest/quickstart/docker.html,] we recommend 
users to expose their docker daemon over TCP.

This is dangerous, because the docker daemon typically has broad system 
permissions, as documented at 
[https://docs.docker.com/engine/security/#docker-daemon-attack-surface]. Making 
the docker daemon available to the Zeppelin service over TCP without 
accidentally also opening it to untrusted clients is hard.

It would be great if the DockerInterpreterProcess could talk to Docker over the 
docker daemon socket: this can be exposed to only the Zeppelin service (and not 
other clients) much easier.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4524: [ZEPPELIN-5859] Upgrading alluxio client library to latest stable release 2.9.0

2022-12-02 Thread GitBox


jongyoul commented on code in PR #4524:
URL: https://github.com/apache/zeppelin/pull/4524#discussion_r1038329132


##
alluxio/src/main/java/org/apache/zeppelin/alluxio/AlluxioInterpreter.java:
##
@@ -74,19 +74,25 @@ public AlluxioInterpreter(Properties property) {
 
 alluxioMasterHostname = property.getProperty(ALLUXIO_MASTER_HOSTNAME);
 alluxioMasterPort = property.getProperty(ALLUXIO_MASTER_PORT);
+
+

Review Comment:
   ```suggestion
   ```



-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul commented on a diff in pull request #4524: [ZEPPELIN-5859] Upgrading alluxio client library to latest stable release 2.9.0

2022-12-02 Thread GitBox


jongyoul commented on code in PR #4524:
URL: https://github.com/apache/zeppelin/pull/4524#discussion_r1038329356


##
alluxio/src/main/java/org/apache/zeppelin/alluxio/AlluxioInterpreter.java:
##
@@ -74,19 +74,25 @@ public AlluxioInterpreter(Properties property) {
 
 alluxioMasterHostname = property.getProperty(ALLUXIO_MASTER_HOSTNAME);
 alluxioMasterPort = property.getProperty(ALLUXIO_MASTER_PORT);
+
+

Review Comment:
   Remove empty lines



-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] raboof opened a new pull request, #4526: [ZEPPELIN-5863] Warn not to expose the docker daemon to untrusted users

2022-12-02 Thread GitBox


raboof opened a new pull request, #4526:
URL: https://github.com/apache/zeppelin/pull/4526

   ### What is this PR for?
   
   This PR adds a warning to the documentation for setting up Zeppelin 
Interpreter on Docker, warning that the docker daemon should not be exposed to 
untrusted users.
   
   ### What type of PR is it?
   Improvement
   Documentation
   
   ### What is the Jira issue?
   
   [ZEPPELIN-5863](https://issues.apache.org/jira/browse/ZEPPELIN-5863)
   
   ### Questions:
   * Does the licenses files need to update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? N/a
   


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[jira] [Created] (ZEPPELIN-5863) Warn not to expose the docker daemon to untrusted users

2022-12-02 Thread Arnout Engelen (Jira)
Arnout Engelen created ZEPPELIN-5863:


 Summary: Warn not to expose the docker daemon to untrusted users
 Key: ZEPPELIN-5863
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-5863
 Project: Zeppelin
  Issue Type: Improvement
  Components: docker, documentation
Reporter: Arnout Engelen


Exposing the docker daemon is dangerous as documented at 
[https://docs.docker.com/engine/security/#docker-daemon-attack-surface] .

 

Since we recommend people expose the docker daemon in 
[https://zeppelin.apache.org/docs/latest/quickstart/docker.html,] we should 
warn people of the possible consequences. I plan to provide a PR to that 
effect..



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[GitHub] [zeppelin] infoankitp commented on pull request #4524: [ZEPPELIN-5859] Upgrading alluxio client library to latest stable release 2.9.0

2022-12-02 Thread GitBox


infoankitp commented on PR #4524:
URL: https://github.com/apache/zeppelin/pull/4524#issuecomment-1335556336

   @jongyoul We are targeting to make zeppelin as the unified shell environment 
and we are using alluxio as well. Was facing problems coz of older client 
version, would be able to productionize it probably when the next version has 
rolled out.  


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul merged pull request #4521: [MINOR] Use Maven CLI option instead of a logger adjustment.

2022-12-02 Thread GitBox


jongyoul merged PR #4521:
URL: https://github.com/apache/zeppelin/pull/4521


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] dependabot[bot] opened a new pull request, #4527: Bump decode-uri-component from 0.2.0 to 0.2.2 in /zeppelin-web-angular

2022-12-02 Thread GitBox


dependabot[bot] opened a new pull request, #4527:
URL: https://github.com/apache/zeppelin/pull/4527

   Bumps 
[decode-uri-component](https://github.com/SamVerschueren/decode-uri-component) 
from 0.2.0 to 0.2.2.
   
   Release notes
   Sourced from https://github.com/SamVerschueren/decode-uri-component/releases";>decode-uri-component's
 releases.
   
   v0.2.2
   
   Prevent overwriting previously decoded tokens  980e0bf
   
   https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.1...v0.2.2";>https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.1...v0.2.2
   v0.2.1
   
   Switch to GitHub workflows  76abc93
   Fix issue where decode throws - fixes https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/6";>#6
  746ca5d
   Update license (https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/1";>#1)
  486d7e2
   Tidelift tasks  a650457
   Meta tweaks  66e1c28
   
   https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.0...v0.2.1";>https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.0...v0.2.1
   
   
   
   Commits
   
   https://github.com/SamVerschueren/decode-uri-component/commit/a0eea469d26eb0df668b081672cdb9581feb78eb";>a0eea46
 0.2.2
   https://github.com/SamVerschueren/decode-uri-component/commit/980e0bf09b64d94f1aa79012f895816c30ffd152";>980e0bf
 Prevent overwriting previously decoded tokens
   https://github.com/SamVerschueren/decode-uri-component/commit/3c8a373dd4837e89b3f970e01295dd03e1405a33";>3c8a373
 0.2.1
   https://github.com/SamVerschueren/decode-uri-component/commit/76abc939783fe3900fadb7d384a74d324d5557f3";>76abc93
 Switch to GitHub workflows
   https://github.com/SamVerschueren/decode-uri-component/commit/746ca5dcb6667c5d364e782d53c542830e4c10b9";>746ca5d
 Fix issue where decode throws - fixes https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/6";>#6
   https://github.com/SamVerschueren/decode-uri-component/commit/486d7e26d3a8c0fbe860fb651fe1bc98c2f2be30";>486d7e2
 Update license (https://github-redirect.dependabot.com/SamVerschueren/decode-uri-component/issues/1";>#1)
   https://github.com/SamVerschueren/decode-uri-component/commit/a65045724e6234acef87f31da499d4807b20b134";>a650457
 Tidelift tasks
   https://github.com/SamVerschueren/decode-uri-component/commit/66e1c2834c0e189201cb65196ec3101372459b02";>66e1c28
 Meta tweaks
   See full diff in https://github.com/SamVerschueren/decode-uri-component/compare/v0.2.0...v0.2.2";>compare
 view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=decode-uri-component&package-manager=npm_and_yarn&previous-version=0.2.0&new-version=0.2.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   - `@dependabot use these labels` will set the current labels as the default 
for future PRs for this repo and language
   - `@dependabot use these reviewers` will set the current reviewers as the 
default for future PRs for this repo and language
   - `@dependabot use these assignees` will set the current assignees as the 
default for future PRs for this repo and language
   - `@dependabot use this milestone` will set the current milestone as the 
default for future PRs for this repo and language
   
   You can disable automated security fix PRs for this re

[GitHub] [zeppelin] infoankitp commented on pull request #4524: [ZEPPELIN-5859] Upgrading alluxio client library to latest stable release 2.9.0

2022-12-02 Thread GitBox


infoankitp commented on PR #4524:
URL: https://github.com/apache/zeppelin/pull/4524#issuecomment-1335623627

   @jongyoul Looks like I can't merge the PR, can you please help me do the 
same or provide me with the access to do so. Thanks ! 


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] Reamer commented on a diff in pull request #4526: [ZEPPELIN-5863] Warn not to expose the docker daemon to untrusted users

2022-12-02 Thread GitBox


Reamer commented on code in PR #4526:
URL: https://github.com/apache/zeppelin/pull/4526#discussion_r1038495622


##
docs/quickstart/docker.md:
##
@@ -55,6 +55,15 @@ vi `/etc/docker/daemon.json`, Add `tcp://0.0.0.0:2375` to 
the `hosts` configurat
 
 `hosts` property reference: 
https://docs.docker.com/engine/reference/commandline/dockerd/
 
+## Security warning

Review Comment:
   Maybe `` instead of `##`?
   What do you think?



-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul commented on pull request #4524: [ZEPPELIN-5859] Upgrading alluxio client library to latest stable release 2.9.0

2022-12-02 Thread GitBox


jongyoul commented on PR #4524:
URL: https://github.com/apache/zeppelin/pull/4524#issuecomment-1336004393

   @infoankitp Apache community has several roles and some roles can have write 
access for repositories only, so I cannot give you individual permission, 
unfortunately. I think 
https://www.apache.org/foundation/how-it-works.html#roles can help you 
understand our policy. However, we are trying to follow up on all contributions 
enthusiastically and you can always ping me and other committers if you want 
help from us. BTW, thank you for your first contribution. We have a lot of 
issues that you can help so we are very welcome to contribute any type of 
contributions including implementing new features, fixing bugs, improving our 
CI system, and so on.
   
   Concerning this PR, some tests failed but it seems unrelated to your 
contribution. Let me merge it.


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul merged pull request #4524: [ZEPPELIN-5859] Upgrading alluxio client library to latest stable release 2.9.0

2022-12-02 Thread GitBox


jongyoul merged PR #4524:
URL: https://github.com/apache/zeppelin/pull/4524


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul opened a new pull request, #4528: [MINOR] Add github autolink to ZEPPELIN

2022-12-02 Thread GitBox


jongyoul opened a new pull request, #4528:
URL: https://github.com/apache/zeppelin/pull/4528

   ### What is this PR for?
   Enable auto link feature to connect with issues.a.o/ZEPPELIN.
   
   
   ### What type of PR is it?
   Improvement
   
   ### Todos
   * [x] - Add `autolink_jira` field to .asf.yaml
   
   ### What is the Jira issue?
   N/A
   
   ### How should this be tested?
   Ppl can click the issue number like ZEPPELIN-
   
   ### Questions:
   * Does the licenses files need to update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No
   


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] infoankitp commented on pull request #4524: [ZEPPELIN-5859] Upgrading alluxio client library to latest stable release 2.9.0

2022-12-02 Thread GitBox


infoankitp commented on PR #4524:
URL: https://github.com/apache/zeppelin/pull/4524#issuecomment-1336040393

   @jongyoul Thanks for making me aware of this ! Well noted ! Let me pick up 
other tickets as well. 


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [zeppelin] jongyoul merged pull request #4528: [MINOR] Add github autolink to ZEPPELIN

2022-12-02 Thread GitBox


jongyoul merged PR #4528:
URL: https://github.com/apache/zeppelin/pull/4528


-- 
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 to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org