[GitHub] incubator-zeppelin pull request: [ZEPPELIN-840] Scalding interpret...

[bzz]

Github user bzz commented on the pull request:

    https://github.com/apache/incubator-zeppelin/pull/917#issuecomment-221789599
  
    Great improvement!
    
    Quick question - is there a reason you prefer to keep some classes in ` 
scalding/src/main/scala/com/twitter/scalding/` vs ` 
scalding/src/main/scala/org/apache/zeppelin/scalding` ?
    
    Both CI failures happen on some tests, like I have not seen before
    ```
    org.apache.zeppelin.notebook.repo.DummyNotebookRepo notebook storage class
    java.lang.ClassNotFoundException: 
org.apache.zeppelin.notebook.repo.DummyNotebookRepo
    
    and
    
    ZeppelinIT.testSparkInterpreterDependencyLoading:209 » ElementNotVisible 
Eleme...
    ```
    Seems not related but definitely deserves a JIRA with `flaky-test` label so 
we could look more into it


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---