You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@zeppelin.apache.org by mo...@apache.org on 2016/06/04 04:31:50 UTC

incubator-zeppelin git commit: [ZEPPELIN-926] Set maxClassfileName to 128

Repository: incubator-zeppelin
Updated Branches:
  refs/heads/master bdc78da0e -> afb1214c2


[ZEPPELIN-926] Set maxClassfileName to 128

### What is this PR for?
set scala Settings.numClassFileSetting to 128 to prevent "Getting 'File name too long'" error.
See https://issues.apache.org/jira/browse/SPARK-4820

### What type of PR is it?
Bug Fix

### Todos
* [x] - Programatically set numClassFileSetting to 128

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-926
https://issues.apache.org/jira/browse/SPARK-4820

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Lee moon soo <mo...@apache.org>

Closes #935 from Leemoonsoo/maxClassfileName and squashes the following commits:

596a33a [Lee moon soo] Set maxClassfileName to 128


Project: http://git-wip-us.apache.org/repos/asf/incubator-zeppelin/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-zeppelin/commit/afb1214c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-zeppelin/tree/afb1214c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-zeppelin/diff/afb1214c

Branch: refs/heads/master
Commit: afb1214c2eefa931dde7270d1215a5ab327f1f61
Parents: bdc78da
Author: Lee moon soo <mo...@apache.org>
Authored: Fri May 27 13:11:43 2016 -0700
Committer: Lee moon soo <mo...@apache.org>
Committed: Fri Jun 3 21:33:03 2016 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/zeppelin/spark/SparkInterpreter.java | 7 +++++++
 1 file changed, 7 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-zeppelin/blob/afb1214c/spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java
----------------------------------------------------------------------
diff --git a/spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java b/spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java
index 0127914..c803cfe 100644
--- a/spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java
+++ b/spark/src/main/java/org/apache/zeppelin/spark/SparkInterpreter.java
@@ -70,6 +70,7 @@ import scala.reflect.io.AbstractFile;
 import scala.tools.nsc.Settings;
 import scala.tools.nsc.interpreter.Completion.Candidates;
 import scala.tools.nsc.interpreter.Completion.ScalaCompleter;
+import scala.tools.nsc.settings.MutableSettings;
 import scala.tools.nsc.settings.MutableSettings.BooleanSetting;
 import scala.tools.nsc.settings.MutableSettings.PathSetting;
 
@@ -462,6 +463,12 @@ public class SparkInterpreter extends Interpreter {
 
     System.setProperty("scala.repl.name.line", "line" + this.hashCode() + "$");
 
+    // To prevent 'File name too long' error on some file system.
+    MutableSettings.IntSetting numClassFileSetting = settings.maxClassfileName();
+    numClassFileSetting.v_$eq(128);
+    settings.scala$tools$nsc$settings$ScalaSettings$_setter_$maxClassfileName_$eq(
+        numClassFileSetting);
+
     synchronized (sharedInterpreterLock) {
       /* create scala repl */
       if (printREPLOutput()) {