[GitHub] zeppelin issue #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-19 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
I doubted CI's test in this PR, so I tried this test for absolutely same 
code in my own repository. And it shows all green.

[travis-ci.org/hyonzin/zeppelin/builds/168996633](https://travis-ci.org/hyonzin/zeppelin/builds/168996633)
I guess all checks are failed in this PR because of cache.


---
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.
---


[GitHub] zeppelin issue #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-18 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@hyonzin Sometimes, CI tests are flaky. Could you re-trigger CI once again 
and see if it goes to green?


---
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.
---


[GitHub] zeppelin issue #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-18 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
Test in SCALA_VER="2.10" SPARK_VER="1.5.2" still fails. I checked its 
[LOG](https://s3.amazonaws.com/archive.travis-ci.org/jobs/168614710/log.txt),

```bash
13:50:41,164  INFO org.apache.zeppelin.rest.AbstractTestRestApi:299 - 200 - 
OK
13:50:41,164  INFO org.apache.zeppelin.rest.AbstractTestRestApi:140 - Test 
Zeppelin stared.
SPARK HOME detected null
```
```bash
+cp spark-1.5.2-bin-hadoop2.3.tgz ..
+cd ..
+tar zxf spark-1.5.2-bin-hadoop2.3.tgz

gzip: stdin: unexpected end of file
tar: Unexpected EOF in archive
tar: Unexpected EOF in archive
tar: Error is not recoverable: exiting now
+echo 'Unable to extract spark-1.5.2-bin-hadoop2.3.tgz'
Unable to extract spark-1.5.2-bin-hadoop2.3.tgz
+rm -rf spark-1.5.2-bin-hadoop2.3
+rm -f spark-1.5.2-bin-hadoop2.3.tgz
+set +xe
```

It seems to be disabled to extract .tgz file, so it can't find Spark Home.
And I wonder why it's associated with my modification yet.


---
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.
---


[GitHub] zeppelin issue #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-18 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@minahlee Thank you for your review! I modified more lines you commented, 
and some trivial placeholder because I think it's better to read. Is it okay? 


---
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.
---


[GitHub] zeppelin issue #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-14 Thread hyonzin
Github user hyonzin commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@jongyoul thank you for review. Yes, and I'm still working in process. I'll 
fix more files.


---
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.
---


[GitHub] zeppelin issue #1518: [ZEPPELIN-1549][WIP] Change NotebookID variable name t...

2016-10-14 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1518
  
@hyonzin @cloverhearts Do you change these two files in this PR?


---
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.
---