[zeppelin] branch master updated: [ZEPPELIN-5539] UnsupportedOperationException in YarnRemoteInterpreterProcess

[jo...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jongyoul pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/zeppelin.git


The following commit(s) were added to refs/heads/master by this push:
     new 7e7ba02  [ZEPPELIN-5539] UnsupportedOperationException in YarnRemoteInterpreterProcess
7e7ba02 is described below

commit 7e7ba0277e2ece0a968c54d482bd45be305a0d6e
Author: Jeff Zhang <zj...@apache.org>
AuthorDate: Tue Sep 28 16:45:43 2021 +0800

    [ZEPPELIN-5539] UnsupportedOperationException in YarnRemoteInterpreterProcess
    
    ### What is this PR for?
    
    The root cause is that `Arrays.asList` return a fixed size List, this PR use `ArrayList` instead to fix this issue.
    
    ### What type of PR is it?
    [Bug Fix ]
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    * https://issues.apache.org/jira/browse/ZEPPELIN-5539
    
    ### How should this be tested?
    * CI pass
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update? No
    * Is there breaking changes for older versions? No
    * Does this needs documentation? No
    
    Author: Jeff Zhang <zj...@apache.org>
    
    Closes #4241 from zjffdu/ZEPPELIN-5539 and squashes the following commits:
    
    187e6e253 [Jeff Zhang] [ZEPPELIN-5539] UnsupportedOperationException in YarnRemoteInterpreterProcess
---
 .../zeppelin/interpreter/launcher/YarnRemoteInterpreterProcess.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/zeppelin-plugins/launcher/yarn/src/main/java/org/apache/zeppelin/interpreter/launcher/YarnRemoteInterpreterProcess.java b/zeppelin-plugins/launcher/yarn/src/main/java/org/apache/zeppelin/interpreter/launcher/YarnRemoteInterpreterProcess.java
index 5aa1a5c..33bca6c 100644
--- a/zeppelin-plugins/launcher/yarn/src/main/java/org/apache/zeppelin/interpreter/launcher/YarnRemoteInterpreterProcess.java
+++ b/zeppelin-plugins/launcher/yarn/src/main/java/org/apache/zeppelin/interpreter/launcher/YarnRemoteInterpreterProcess.java
@@ -372,7 +372,7 @@ public class YarnRemoteInterpreterProcess extends RemoteInterpreterProcess {
    * classpath specified through the Hadoop and Yarn configurations.
    */
   private void populateHadoopClasspath(Map<String, String> envs) {
-    List<String> yarnClassPath = Arrays.asList(getYarnAppClasspath());
+    List<String> yarnClassPath = new ArrayList(Arrays.asList(getYarnAppClasspath()));
     List<String> mrClassPath = Arrays.asList(getMRAppClasspath());
     yarnClassPath.addAll(mrClassPath);
     if (LOGGER.isInfoEnabled()) {