[GitHub] zeppelin pull request #3241: ZEPPELIN-3880: Code refactoring: remove class f...

[monsieurp <gi...@git.apache.org>]

GitHub user monsieurp opened a pull request:

    https://github.com/apache/zeppelin/pull/3241

    ZEPPELIN-3880: Code refactoring: remove class files using a Java stream

    ### What is this PR for?
    Make use of a Java stream instead of an ugly for-loop to get rid of class files.
    
    ### What type of PR is it?
    Improvement
    
    ### What is the Jira issue?
    https://issues.apache.org/jira/browse/ZEPPELIN-3880
    
    ### How should this be tested?
    * CI pass.
    
    ### Questions:
    * Does the licenses files need update? No.
    * Is there breaking changes for older versions? No.
    * Does this needs documentation? No.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/monsieurp/zeppelin ZEPPELIN-3880

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/zeppelin/pull/3241.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3241
    
----
commit 2b11f4f6ffc5f5478c4cc887659bd294631188c7
Author: Patrice Clement <mo...@...>
Date:   2018-11-24T22:02:42Z

    ZEPPELIN-3880: Code refactoring: remove class files using a Java stream

----


---

[GitHub] zeppelin issue #3241: ZEPPELIN-3880: Code refactoring: remove class files us...

[felixcheung <gi...@git.apache.org>]

Github user felixcheung commented on the issue:

    https://github.com/apache/zeppelin/pull/3241
  
    tbh, this seems rather minor. I'd leave this for others to comment.


---