[zeppelin] branch master updated: [ZEPPELIN-4320] fix packages to bundle list

[mo...@apache.org]

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

moon 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 a69d284  [ZEPPELIN-4320] fix packages to bundle list
a69d284 is described below

commit a69d284e4552f961809f6ce11aca95fd732bc8af
Author: santosh1994 <sa...@gmail.com>
AuthorDate: Wed Oct 16 10:49:51 2019 +0530

    [ZEPPELIN-4320] fix packages to bundle list
    
    ### What is this PR for?
    Enabled Helium packages are not getting installed/bundled on zeppelin startup. Instead getting bundled later on paragraph with bundle dependency are loaded.
    
    ### What type of PR is it?
    Bug Fix
    
    ### Todos
    * [ ] - Task
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-4320
    
    ### How should this be tested?
    * Enable some helium packages
    * Remove helium cache folder(local-repo)
    * Restart zeppelin and verify, enabled packages are getting bundled on zeppelin server start
    
    ### Screenshots (if appropriate)
    
    ### Questions:
    * Does the licenses files need update?
    * Is there breaking changes for older versions?
    * Does this needs documentation?
    
    Author: santosh1994 <sa...@gmail.com>
    
    Closes #3438 from santosh1994/ZEPPELIN-4320 and squashes the following commits:
    
    a805d4d70 [santosh1994] [ZEPPELIN-4320] fix packages to bundle list
---
 zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/Helium.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/Helium.java b/zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/Helium.java
index e0a3845..9bd3bce 100644
--- a/zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/Helium.java
+++ b/zeppelin-zengine/src/main/java/org/apache/zeppelin/helium/Helium.java
@@ -454,7 +454,7 @@ public class Helium {
         }
       }
     }
-    new LinkedList<>().addAll(orderedBundlePackages);
+    output.addAll(orderedBundlePackages);
     return output;
   }