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

   ### What is this PR for?
   [Powermock](https://github.com/powermock/powermock/issues/929) is not 
supported in JUnit5. We should not be dependent on Powermock. This PR removes 
the Powermock dependency from the Docker plugin. It also makes some refactoring.
   
   Instead of using environment variables directly, this possibility of 
configuration is moved to ZeppelinConfiguration. In order not to break the 
naming scheme in ZeppelinConfiguration, the environment variables that were 
previously accessed must be changed.
   
   `CONTAINER_SPARK_HOME` -> `ZEPPELIN_DOCKER_CONTAINER_SPARK_HOME`
   `UPLOAD_LOCAL_LIB_TO_CONTAINTER` -> 
`ZEPPELIN_DOCKER_UPLOAD_LOCAL_LIB_TO_CONTAINTER`
   `DOCKER_HOST` -> `ZEPPELIN_DOCKER_HOST`
   `DOCKER_TIME_ZONE` -> `ZEPPELIN_DOCKER_TIME_ZONE`
   
   Except for DOCKER_TIME_ZONE, I have not found any references in the 
documentation or other files.
   
   ### What type of PR is it?
   
   Refactoring
   
   ### What is the Jira issue?
   * https://issues.apache.org/jira/browse/ZEPPELIN-5855
   
   ### How should this be tested?
   * CI
   
   ### Questions:
   * Does the licenses files need to update? No
   * Is there breaking changes for older versions? Yes
   * 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

Reply via email to