Skip to content

Conversation

@michrawson
Copy link
Contributor

What is this PR for?

Don't plot 0 for unknown data.
And
Allow for custom location for interpreter.json.
Try to create dir if needed.
Die if file or file's dir not writeable.

What type of PR is it?

Bug Fix

How should this be tested?

  1. Test that D3 plots are correct.
  2. Test default: when not using a custom file, everything should work as usual.
  3. Specify any file via zeppelin-site.xml AND zeppelin-env.sh and that file should be used.
  4. If the file exists and is not writeable, zeppelin shouldn't start.
  5. If the file's dir doesn't exit, try to create it.
  6. If the file doesn't exist and the file's dir is not writeable, zeppelin shouldn't start.

Questions:

  • Does the licenses files need update?
    No
  • Is there breaking changes for older versions?
    No
  • Does this need documentation?
    Added in docs/

@Leemoonsoo
Copy link
Member

Thanks for contribution. Some tests are failing after this change. Could you take a look CI logs?

@michrawson
Copy link
Contributor Author

Failed due to network or npmjs.org or encryption or certs etc. Not because of this PR.

[ERROR] npm http GET https://registry.npmjs.org/grunt-angular-templates
[INFO] BUILD FAILURE

@Leemoonsoo
Copy link
Member

Could you check https://travis-ci.org/apache/incubator-zeppelin/jobs/115949452 ? if i search log file for "reactor summary" i got following logs

Results :

Tests in error: 
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NotebookTest.setUp:86 » IO No such file or directory
  NoteInterpreterLoaderTest.setUp:64 » IO No such file or directory
  NoteInterpreterLoaderTest.setUp:64 » IO No such file or directory
  NotebookRepoSyncTest.setUp:95 » IO No such file or directory
  NotebookRepoSyncTest.setUp:95 » IO No such file or directory
  NotebookRepoSyncTest.setUp:95 » IO No such file or directory
  NotebookRepoSyncTest.setUp:95 » IO No such file or directory
  NotebookRepoSyncTest.setUp:95 » IO No such file or directory
  NotebookRepoSyncTest.setUp:95 » IO No such file or directory
  VFSNotebookRepoTest.setUp:79 » IO No such file or directory
  InterpreterFactoryTest.setUp:63 » IO No such file or directory
  InterpreterFactoryTest.setUp:63 » IO No such file or directory
  InterpreterFactoryTest.setUp:63 » IO No such file or directory
  InterpreterFactoryTest.setUp:63 » IO No such file or directory

Tests run: 63, Failures: 0, Errors: 29, Skipped: 0

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Zeppelin ........................................... SUCCESS [ 18.611 s]
[INFO] Zeppelin: Interpreter .............................. SUCCESS [01:44 min]
[INFO] Zeppelin: Zengine .................................. FAILURE [  8.090 s]
[INFO] Zeppelin: Display system apis ...................... SKIPPED
[INFO] Zeppelin: Spark dependencies ....................... SKIPPED
[INFO] Zeppelin: Spark .................................... SKIPPED
[INFO] Zeppelin: Markdown interpreter ..................... SKIPPED
[INFO] Zeppelin: Angular interpreter ...................... SKIPPED
[INFO] Zeppelin: Shell interpreter ........................ SKIPPED
[INFO] Zeppelin: Hive interpreter ......................... SKIPPED
[INFO] Zeppelin: HBase interpreter ........................ SKIPPED
[INFO] Zeppelin: Apache Phoenix Interpreter ............... SKIPPED
[INFO] Zeppelin: PostgreSQL interpreter ................... SKIPPED
[INFO] Zeppelin: JDBC interpreter ......................... SKIPPED
[INFO] Zeppelin: Tajo interpreter ......................... SKIPPED
[INFO] Zeppelin: Flink .................................... SKIPPED
[INFO] Zeppelin: Apache Ignite interpreter ................ SKIPPED
[INFO] Zeppelin: Kylin interpreter ........................ SKIPPED
[INFO] Zeppelin: Lens interpreter ......................... SKIPPED
[INFO] Zeppelin: Cassandra ................................ SKIPPED
[INFO] Zeppelin: Elasticsearch interpreter ................ SKIPPED
[INFO] Zeppelin: Alluxio interpreter ...................... SKIPPED
[INFO] Zeppelin: web Application .......................... SKIPPED
[INFO] Zeppelin: Server ................................... SKIPPED
[INFO] Zeppelin: Packaging distribution ................... SKIPPED
[INFO] Zeppelin: Scalding interpreter ..................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 02:11 min
[INFO] Finished at: 2016-03-14T19:48:15+00:00
[INFO] Final Memory: 36M/472M
[INFO] ------------------------------------------------------------------------

@michrawson michrawson changed the title Allow custom location for interpreter.json and die if not writeable D3 Plotting change and Allow custom location for interpreter.json and die if not writeable Mar 18, 2016
@michrawson
Copy link
Contributor Author

How do I get this test to pass?

FAILURE! - in org.apache.zeppelin.dep.DependencyResolverTest
testLoad(org.apache.zeppelin.dep.DependencyResolverTest) Time elapsed: 15.664 sec <<< ERROR!
java.lang.NullPointerException: null
at org.sonatype.aether.impl.internal.DefaultRepositorySystem.resolveDependencies(DefaultRepositorySystem.java:352)
at org.apache.zeppelin.dep.DependencyResolver.getArtifactsWithDep(DependencyResolver.java:170)
at org.apache.zeppelin.dep.DependencyResolver.loadFromMvn(DependencyResolver.java:123)
at org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:81)
at org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:94)
at org.apache.zeppelin.dep.DependencyResolver.load(DependencyResolver.java:68)
at org.apache.zeppelin.dep.DependencyResolverTest.testLoad(DependencyResolverTest.java:80)

@felixcheung
Copy link
Member

Seems like this PR should be split into 2 JIRAs? It would be better to track changes this way.

@corneadoug
Copy link
Contributor

@michrawson Could you split this PR in two?

@asfgit asfgit closed this in c38a0a0 May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close #83
close #86
close #125
close #133
close #139
close #146
close #193
close #203
close #246
close #262
close #264
close #273
close #291
close #299
close #320
close #347
close #389
close #413
close #423
close #543
close #560
close #658
close #670
close #728
close #765
close #777
close #782
close #783
close #812
close #822
close #841
close #843
close #878
close #884
close #918
close #989
close #1076
close #1135
close #1187
close #1231
close #1304
close #1316
close #1361
close #1385
close #1390
close #1414
close #1422
close #1425
close #1447
close #1458
close #1466
close #1485
close #1492
close #1495
close #1497
close #1536
close #1545
close #1561
close #1577
close #1600
close #1603
close #1678
close #1695
close #1739
close #1748
close #1765
close #1767
close #1776
close #1783
close #1799
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants