from:"astroshim"

[GitHub] incubator-zeppelin pull request #964: filtering subdirectory names.

2016-06-05 Thread astroshim

GitHub user astroshim opened a pull request:

https://github.com/apache/incubator-zeppelin/pull/964

filtering subdirectory names.

### What is this PR for?
It needs to filtering subdirectory names in navbar.


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


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


### How should this be tested?
try filter the note name on navbar.


### Screenshots (if appropriate)

![gif](https://cloud.githubusercontent.com/assets/3348133/15806159/31026fc2-2b78-11e6-936c-562ae4c6.gif)


### 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/astroshim/incubator-zeppelin ZEPPELIN-886

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

https://github.com/apache/incubator-zeppelin/pull/964.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 #964


commit 1167d0d398062387f6f89bd165815088ecddaf93
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-05T14:48:41Z

filtering subdirectory names.




---
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] incubator-zeppelin pull request #979: Alert message when trying to delete la...

2016-06-09 Thread astroshim

GitHub user astroshim opened a pull request:

https://github.com/apache/incubator-zeppelin/pull/979

Alert message when trying to delete last paragraph.

### What is this PR for?
This PR is for giving proper information to users and blocking unnecessary 
communication with zeppelin server when trying to delete last paragraph because 
the last paragraph can't be deleted.


### What type of PR is it?
Improvement 


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


### How should this be tested?
You can try to remove last paragraph of a notebook.


### Screenshots (if appropriate)

![image](https://cloud.githubusercontent.com/assets/3348133/15929252/0edaa6d2-2e87-11e6-9baa-8f2b5f63d592.png)



### 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/astroshim/incubator-zeppelin ZEPPELIN-979

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

https://github.com/apache/incubator-zeppelin/pull/979.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 #979


commit faa7067c9db8bf34cceaac08d46d42bdda24cd8f
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-09T12:03:17Z

alert message when trying to delete last paragraph.




---
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] incubator-zeppelin issue #919: support changing tabsize of paragraph

2016-06-09 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/incubator-zeppelin/pull/919
  
@corneadoug Thank you for your concern! I'm thinking to find right way how 
to handle this issue.


---
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 #979: Alert message when trying to delete last paragraph.

2016-06-13 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
@corneadoug Thank you for your kind explanation. 
I know the reason about the error but I couldn't reproduce the error even 
if i build like ```mvn clean package```. 
Did you build also like i did?


---
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 #979: Alert message when trying to delete last paragraph.

2016-06-13 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
@corneadoug Okay. Thank you!!


---
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 #979: Alert message when trying to delete last paragraph.

2016-06-12 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
@minahlee Could you tell me how can i get your test failure?


---
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 #964: filtering subdirectory names.

2016-06-10 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/964
  
@corneadoug I refactored the codes that you pointed out. please review.


---
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 #979: Alert message when trying to delete last paragraph.

2016-06-10 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
@corneadoug @minahlee I fixed the code to hide "Remove" menu on the last 
paragraph menu. please review.


---
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 #964: filtering subdirectory names.

2016-06-10 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/964
  
@corneadoug Thank you for your review.
I'll refactor the code as you mentioned. 


---
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] incubator-zeppelin issue #959: [ZEPPELIN-951] Update cron expression link an...

2016-06-03 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/incubator-zeppelin/pull/959
  
LGTM


---
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 #965: [ZEPPELIN-957] Normalize note name.

2016-06-11 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/965
  
I really appreciate your detailed review. @Leemoonsoo 
I will fix 1,2,3 but in case 4, 
I thought ```/MyNote/sub``` and ```MyNote/sub``` are the same from 
directory point of view.
Do you think they should be treated differently?


---
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 #1011: Update licenses

2016-06-14 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1011
  
LGTM


---
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 #976: ZEPPLIN-976 ] HotFix - zeppelin server does not restart...

2016-06-14 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/976
  
LGTM. 
@minahlee The selenium test of travis has a problem so @corneadoug is 
fixing now.


---
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 #979: Alert message when trying to delete last paragraph.

2016-06-14 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/979
  
@corneadoug I really appreciate for your effort. Thank you. I reset my 
commits.


---
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] incubator-zeppelin pull request #965: [ZEPPELIN-957] Normalize note name.

2016-06-05 Thread astroshim

GitHub user astroshim opened a pull request:

https://github.com/apache/incubator-zeppelin/pull/965

[ZEPPELIN-957] Normalize note name.

### What is this PR for?
It's better if notebook name can be normalized.


### What type of PR is it?
Improvement


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


### How should this be tested?
Try create notebook name with many slashes and no slashes.
You can refer to screen shot.


### Screenshots (if appropriate)
  - before

![be](https://cloud.githubusercontent.com/assets/3348133/15809552/9bf9e26e-2bce-11e6-942c-df655ee52176.gif)


  - after

![af](https://cloud.githubusercontent.com/assets/3348133/15809553/9ff43c48-2bce-11e6-9f69-2b7a00bfc91e.gif)


### 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/astroshim/incubator-zeppelin ZEPPELIN-957

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

https://github.com/apache/incubator-zeppelin/pull/965.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 #965


commit e5499772475275c178790d473b42c5d6add3b1b3
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-06T00:59:51Z

normalize note name




---
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] incubator-zeppelin issue #972: ZEPPELIN-963 ] Jobmanagement - (1) basic back...

2016-06-08 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/incubator-zeppelin/pull/972
  
@cloverhearts Thank you for your explanation. You mean that the 
"LastResponseUnixTime" is for getting notebook information of lastest updated.


---
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] incubator-zeppelin issue #972: ZEPPELIN-963 ] Jobmanagement - (1) basic back...

2016-06-08 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/incubator-zeppelin/pull/972
  
It's working nicely.
but Is the correct behavior that the lastResponseUnixTime is increased 
whenever call the curl?
Then i can't understand what's the purpose of step 3. Could you tell me 
more detail?



---
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] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-25 Thread astroshim

GitHub user astroshim reopened a pull request:

https://github.com/apache/incubator-zeppelin/pull/912

Hotfix - does not working notebook filter on navibar.

### What is this PR for?
It doesn't working the notebook filter on navibar.


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


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


### Screenshots (if appropriate)
- before 

![image](https://cloud.githubusercontent.com/assets/3348133/15497775/405467f0-21d7-11e6-9eb1-789b21f1764c.png)


- after

![image](https://cloud.githubusercontent.com/assets/3348133/15497786/4cc02966-21d7-11e6-855c-3138d617e4c1.png)



### 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/astroshim/incubator-zeppelin ZEPPELIN-873

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

https://github.com/apache/incubator-zeppelin/pull/912.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 #912


commit 969aa07d1682b9b0d195a01b7b3ffe035d46e781
Author: astroshim <hss...@nflabs.com>
Date:   2016-05-24T08:37:54Z

fix bug

commit 0cfed636f56d25baef4f815b6e482d95b80d16ef
Author: astroshim <hss...@nflabs.com>
Date:   2016-05-25T08:32:14Z

support folder structure in navbar filter.




---
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] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-25 Thread astroshim

Github user astroshim closed the pull request at:

https://github.com/apache/incubator-zeppelin/pull/912


---
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] incubator-zeppelin pull request: ZEPPELIN-885 Add option to send e...

2016-05-25 Thread astroshim

Github user astroshim commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/918#issuecomment-221772165
  
@cloverhearts maybe you need to set up the smtp informations. you can set 
the variables in the "conf/zeppelin-site.xml".


---
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] incubator-zeppelin pull request: support changing tabsize of parag...

2016-05-27 Thread astroshim

Github user astroshim commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/919#issuecomment-222195524
  
@Leemoonsoo It really makes sense and I thought that too. 
The reason why I thought about configuring tab size in notebook is better 
was just because easier for the users to set up.
but your example might more make sense.
Thanks.


---
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] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-26 Thread astroshim

Github user astroshim commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-221791290
  
@bzz I made issue(https://issues.apache.org/jira/browse/ZEPPELIN-886). 


---
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] incubator-zeppelin pull request: Hotfix - does not working noteboo...

2016-05-26 Thread astroshim

Github user astroshim commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/912#issuecomment-222044643
  
@minahlee It's my mistake. I reverted.


---
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] incubator-zeppelin pull request: support changing tabsize of parag...

2016-05-26 Thread astroshim

Github user astroshim commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/919#issuecomment-222048031
  
@Leemoonsoo Good point. I agree with you that changing tab size in the 
interpreter or notebook is better than paragraph. so I think Configuring tab 
size of notebook level makes more sense than interpreter level, what do you 
think?


---
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] incubator-zeppelin issue #938: ZEPPELIN-930 ] fix width and not close li tag...

2016-06-01 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/incubator-zeppelin/pull/938
  
LGTM


---
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] incubator-zeppelin pull request: Zeppelin-531 ] Job management

2016-05-29 Thread astroshim

Github user astroshim commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/921#issuecomment-222408542
  
@cloverhearts Really nice job. but I got an follwing error when i click the 
paragraph in job

![image](https://cloud.githubusercontent.com/assets/3348133/15639594/b3852256-2668-11e6-8273-75570455ba56.png)



---
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] incubator-zeppelin pull request: Hotfix - initial notebook name do...

2016-05-29 Thread astroshim

GitHub user astroshim opened a pull request:

https://github.com/apache/incubator-zeppelin/pull/929

Hotfix - initial notebook name does not increase.

### What is this PR for?
To increase initial notebook name.


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


### How should this be tested?
execute create new note.

### Screenshots (if appropriate)
 - before

![image](https://cloud.githubusercontent.com/assets/3348133/15640275/60d1b67e-2671-11e6-9a46-645f28b1a09a.png)


- after

![image](https://cloud.githubusercontent.com/assets/3348133/15640270/47039e92-2671-11e6-9000-75bfaa5fa21a.png)


### 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/astroshim/incubator-zeppelin hotfix/notename

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

https://github.com/apache/incubator-zeppelin/pull/929.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 #929


commit eb0defdb50c5da8a2f562f899a499c66494e6494
Author: astroshim <hss...@nflabs.com>
Date:   2016-05-30T05:13:57Z

list name has changed.




---
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 #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1058
  
The selenium test has passed!!


---
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 pull request #1064: BugFix-blocking of blank values insertion on th...

2016-06-21 Thread astroshim

GitHub user astroshim opened a pull request:

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

BugFix-blocking of blank values insertion on the Credential page.

### What is this PR for?
This PR blocks the blank values insertion on the Credential page and 
changes the success message box to zeppelin's dialog box.


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


### How should this be tested?
Try to save with blank values on the Credential page.

### Screenshots (if appropriate)
  - before 

![out](https://cloud.githubusercontent.com/assets/3348133/16255783/c5d19378-3887-11e6-9eec-5fcac42ee276.gif)


  - after

![image](https://cloud.githubusercontent.com/assets/3348133/16255706/18967912-3887-11e6-8823-21683c17082d.png)



![image](https://cloud.githubusercontent.com/assets/3348133/16255722/55c9f494-3887-11e6-84f8-857a7f5380d6.png)


### 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/astroshim/zeppelin feat/checkCredentialValues

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

https://github.com/apache/zeppelin/pull/1064.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 #1064


commit e148d44e10850c8bab4f0d9c7e3b97b1e48cf7a0
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-22T05:04:26Z

disallow the blank values of Credential on the front.




---
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 #1064: BugFix-blocking of blank values insertion on the Crede...

2016-06-22 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1064
  
@bzz I fixed as your comments. please review.


---
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 #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1058
  
@Leemoonsoo I rebased.


---
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 #1036: [MINOR] Add .Rhistory in .gitignore

2016-06-17 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1036
  
LGTM


---
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 #919: support changing tabsize of paragraph

2016-06-21 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/919
  
Thank you for your detailed explanation @Leemoonsoo and I agree with moving 
interpreter-setting.json.



---
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 #1037: fix typo the description of interpreter menu.

2016-06-19 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1037
  
@minahlee Thanks for pointing out. I just fixed.


---
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 pull request #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion...

2016-06-20 Thread astroshim

GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1033] HotFixing of paragraph deletion and re-ordering broken

### What is this PR for?
This PR is for fixing [this 
issue](https://issues.apache.org/jira/browse/ZEPPELIN-1033)


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


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


### 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/astroshim/zeppelin ZEPPELIN-1033

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

https://github.com/apache/zeppelin/pull/1048.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 #1048


commit 6c99111bdd39e1c76817c9f0fb3a37a9b17bd998
Author: root <r...@worker8.nflabs.com>
Date:   2016-06-20T06:21:45Z

add initSecurityManager method.




---
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 #1058: BugFix of Automatically adds %.* of previous paragraph...

2016-06-22 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1058
  
@Leemoonsoo Sorry for late response. I fixed.


---
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 #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion and re...

2016-06-20 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1048
  
@randerzander Thank you for your review. Let me check.


---
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 #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-20 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1030
  
Thank you for your review @bzz and @AhyoungRyu and @cloverhearts !
And It really makes sense @bzz comments.
Let me fix them.


---
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 #919: support changing tabsize of paragraph

2016-06-20 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/919
  
@bzz Thank you for your detail review! 
I was worring about changing tab size per interpreter as @Leemoonsoo 
suggests is more hard to config than per notebook or per paragraph.
How about implementing per notebook first and then let's talk it's good or 
bad?
and This changing is not affect to paragraphs of existing.


---
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 #1048: [ZEPPELIN-1033] HotFixing of paragraph deletion and re...

2016-06-20 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1048
  
nice @randerzander thanks for addressing this issue and review.


---
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 pull request #1058: BugFix of Automatically adds %.* of previous pa...

2016-06-21 Thread astroshim

GitHub user astroshim opened a pull request:

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

BugFix of Automatically adds %.* of previous paragraph's typing.

### What is this PR for?
This PR is to fix bug from https://github.com/apache/zeppelin/pull/806.
The default interpreter name that automatically adds should be interpreter 
group name. 

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


### Screenshots (if appropriate)
  - before 

![b](https://cloud.githubusercontent.com/assets/3348133/16221424/99cab6b6-37cd-11e6-8074-93fd9db53b45.gif)

  - after

![a](https://cloud.githubusercontent.com/assets/3348133/16221623/d2c36e94-37ce-11e6-957f-fea2e67dd55d.gif)


### 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/astroshim/zeppelin bugfix/ZEPPELIN-707

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

https://github.com/apache/zeppelin/pull/1058.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 #1058


commit a3123da471b7c96942275a509f28c99e78f55ccc
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-21T07:17:58Z

default interpreter name should be group name.




---
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 pull request #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-16 Thread astroshim

GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1023] Add more credential apis.

### What is this PR for?
This PR is for supporting various Credential APIs for users.


### What type of PR is it?
Improvement


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


### How should this be tested?
  - to create credential information.
```
curl -XPUT -H "Content-Type: application/json" 
"http://localhost:8080/api/credential; -d '{"entity" : "e1", "username" : 
"user1", "password" : "testpass"}'
```

  - to get credential information.
```
curl -XGET -H "Content-Type: application/json" 
"http://localhost:8080/api/credential; 
```

  - to remove credential entity information.
```
curl -vvv -XDELETE -H "Content-Type: application/json" 
"http://localhost:8080/api/credential/e1;
```

  - to remove all credential information.
```
curl -vvv -XDELETE -H "Content-Type: application/json" 
"http://localhost:8080/api/credential;
```

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



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

$ git pull https://github.com/astroshim/zeppelin ZEPPELIN-1023

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

https://github.com/apache/zeppelin/pull/1030.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 #1030


commit 03717018b88d4a4d02f0ce153bb006c5474f7dd7
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-16T15:05:42Z

add more credential apis.




---
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 #1079: Prevent menubar hide notebook title and control button...

2016-06-24 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1079
  
LGTM


---
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 #1025: ZEPPELIN-964 ] Jobmanagement - (2) basic front-end

2016-06-17 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1025
  
@cloverhearts Thanks for the quick response. I understand. :+1: 



---
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 pull request #1037: fix typo the description of interpreter menu.

2016-06-17 Thread astroshim

GitHub user astroshim opened a pull request:

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

fix typo the description of interpreter menu.

### What is this PR for?
This PR is for fixing typo of the interpreter menu description.


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


### Screenshots (if appropriate)
  - before
 
![image](https://cloud.githubusercontent.com/assets/3348133/16156875/1dba142e-34f1-11e6-937b-c3d2c75a6c42.png)


  - after

![image](https://cloud.githubusercontent.com/assets/3348133/16156901/365979a2-34f1-11e6-91a7-4c99b78a1d86.png)



### 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/astroshim/zeppelin feat/typoInterpreters

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

https://github.com/apache/zeppelin/pull/1037.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 #1037


commit 64f1f608a38f42eadbee202f037a4d3c5ac6189d
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-17T16:04:24Z

fix typo description of interpreter menu.




---
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 #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-27 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1030
  
@bzz I fixed code and the build problem that you mentioned is gone.
but another following build fail occurred. 
I think following problem seems not related this PR, what do you think? 
```
14:10:13,190 ERROR org.apache.zeppelin.scheduler.RemoteScheduler:280 - 
Unknown status
java.util.ConcurrentModificationException
at 
java.util.LinkedList$ListItr.checkForComodification(LinkedList.java:953)
at java.util.LinkedList$ListItr.next(LinkedList.java:886)
at 
com.google.gson.internal.bind.CollectionTypeAdapterFactory$Adapter.write(CollectionTypeAdapterFactory.java:95)
at 
com.google.gson.internal.bind.CollectionTypeAdapterFactory$Adapter.write(CollectionTypeAdapterFactory.java:60)
at 
com.google.gson.internal.bind.TypeAdapterRuntimeTypeWrapper.write(TypeAdapterRuntimeTypeWrapper.java:68)
at 
com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$1.write(ReflectiveTypeAdapterFactory.java:89)
at 
com.google.gson.internal.bind.ReflectiveTypeAdapterFactory$Adapter.write(ReflectiveTypeAdapterFactory.java:195)
at com.google.gson.Gson.toJson(Gson.java:582)
at com.google.gson.Gson.toJson(Gson.java:561)
at com.google.gson.Gson.toJson(Gson.java:516)
at com.google.gson.Gson.toJson(Gson.java:496)
at 
org.apache.zeppelin.notebook.repo.VFSNotebookRepo.save(VFSNotebookRepo.java:212)
at 
org.apache.zeppelin.notebook.repo.NotebookRepoSync.save(NotebookRepoSync.java:145)
at org.apache.zeppelin.notebook.Note.persist(Note.java:543)
at 
org.apache.zeppelin.socket.NotebookServer$ParagraphListenerImpl.afterStatusChange(NotebookServer.java:1195)
at org.apache.zeppelin.scheduler.Job.setStatus(Job.java:150)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobRunner.afterStatusChange(RemoteScheduler.java:385)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobStatusPoller.getStatus(RemoteScheduler.java:267)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobRunner.run(RemoteScheduler.java:341)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$201(ScheduledThreadPoolExecutor.java:178)
at 
java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:292)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)
14:10:13,191  INFO org.apache.zeppelin.socket.NotebookServer:1192 - Job 
20160627-141012_944285838 is finished
14:10:13,191  INFO org.apache.zeppelin.notebook.Paragraph:252 - run 
paragraph 20160627-141013_1822933698 using r 
org.apache.zeppelin.interpreter.LazyOpenInterpreter@38605c72
```


---
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 pull request #1097: init newRepoSetting values

2016-06-27 Thread astroshim

GitHub user astroshim opened a pull request:

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

init newRepoSetting values

### What is this PR for?
This PR fixes initialize value bug of ```Add New Repository```.


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


### How should this be tested?
1. launch the ```Add New Repository``` modal window.
2. put URL value 
3. hit Cancel button
4. launch the ```Add New Repository``` modal window again.


### Screenshots (if appropriate)
  - before

![bb](https://cloud.githubusercontent.com/assets/3348133/16388909/5fbddd00-3cd6-11e6-9e46-1afbb8a37356.gif)


  - after

![after](https://cloud.githubusercontent.com/assets/3348133/16388911/6294390c-3cd6-11e6-92ae-01a8730e3c25.gif)


### 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/astroshim/zeppelin bugfix/initRepositoryValues

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

https://github.com/apache/zeppelin/pull/1097.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 #1097


commit f5a08d544d91082c807e5a3c0b5cc93b09698708
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-27T17:08:47Z

init newRepoSetting values




---
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 #1018: Blocking the zeppelin-web build error.

2016-06-15 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1018
  
It's not actual error just error messsage.


---
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 #1104: [ZEPPELIN-1078] Does not show the text result in the p...

2016-06-29 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1104
  
re-trigger CI


---
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 pull request #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1078] Does not show the text result in the paragraph sometimes.

### What is this PR for?
This PR fixes the problem that does not show the text result in the 
paragraph sometimes.


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


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


### How should this be tested?
Try run python code constantly like screenshot.

### Screenshots (if appropriate)
  - before

![before](https://cloud.githubusercontent.com/assets/3348133/16436829/214a2b4e-3ddc-11e6-9af2-2ee1d7e2cf96.gif)


### 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/astroshim/zeppelin ZEPPELIN-1078

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

https://github.com/apache/zeppelin/pull/1104.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 #1104


commit 441357c042ed557c63f18bb15b2833293431bd06
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-29T00:28:41Z

remove resultRefreshed value.




---
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 pull request #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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] incubator-zeppelin pull request: ZEPPELIN-598 ] Dynamic loading fo...

2016-05-23 Thread astroshim

Github user astroshim commented on the pull request:

https://github.com/apache/incubator-zeppelin/pull/908#issuecomment-221145936
  
@cloverhearts  I tested and really nice working. Thanks.


---
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 #919: support changing tabsize of paragraph

2016-07-26 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/919
  
https://github.com/apache/zeppelin/pull/1218 is better way.


---
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 pull request #919: support changing tabsize of paragraph

2016-07-26 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 #1110: [ZEPPELIN-1091] Disable ace editor's showSettingsMenu ...

2016-07-12 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1110
  
Is this PR going to be merged?


---
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 #1185: Redirect home when requested a wrong notename path.

2016-07-14 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1185
  
@AhyoungRyu Thank you for your opinion. 
Your idea is great but now is redirected all to home when requested wrong 
path. (you can see the related code 
[here](https://github.com/apache/zeppelin/blob/master/zeppelin-web/src/app/app.js#L85))
so I think it's better to follow the present rule. What do you think?


---
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 pull request #1227: [ZEPPELIN-1198][Spark Standalone] Documents for...

2016-07-25 Thread astroshim

GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1198][Spark Standalone] Documents for running zeppelin on 
production environments.

### What is this PR for?
This PR is for documentation for running zeppelin on production 
environments. 

### What type of PR is it?
Documentation

### Todos
- [x] Spark Standalone 
- [ ] Spark on Mesos
- [ ] Spark on YARN 
- [ ] CDH 
- [ ] EMR

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


### 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/astroshim/zeppelin ZEPPELIN-1198/standalone

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

https://github.com/apache/zeppelin/pull/1227.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 #1227


commit 83fdef6e2b6eec8280b566c33a25ee4de2946c7f
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-25T15:43:59Z

doc for spark standalone




---
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 #1223: [WIP Zeppelin-945] Interpreter authorization

2016-07-25 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1223
  
@prabhjyotsingh Thank you for share your great idea.
This PR is for interpreter authorization but your idea seems include user 
management.
I think it should be separated and I agree with @Leemoonsoo's idea of 
giving the same user experience for the authorization inside of Zeppelin.
and I also agree with @prabhjyotsingh's idea of the difference with 
notebook's because interpreter authorization is n:n. (it's hard to show with 
notebook's way.)
so What do you guys think about following way?

- when you want to grant permission of interpreters to all users 

![image](https://cloud.githubusercontent.com/assets/3348133/17102852/6a7622b6-52b6-11e6-98ec-06f120413ee5.png)

- when you want to grant permission of interpreters to specific user

![image](https://cloud.githubusercontent.com/assets/3348133/17102416/7184d4be-52b4-11e6-87b4-74211dd60c90.png)



---
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 #1223: [WIP Zeppelin-945] Interpreter authorization

2016-07-23 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1223
  
@Leemoonsoo Thank you for your review and I think your opinion makes sense. 
I'll change the UI.


---
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 #1117: [ZEPPELIN-1086] Auto Completion for Interpreter.

2016-07-21 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1117
  
Ping


---
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 pull request #1229: [BugFix] Show checkbox for "Connect to existing...

2016-08-01 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 #1229: [BugFix] Show checkbox for "Connect to existing proces...

2016-08-01 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1229
  
re-trigger CI


---
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 pull request #1229: [BugFix] Show checkbox for "Connect to existing...

2016-08-01 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[BugFix] Show checkbox for "Connect to existing process" on interpreter 
menu.

### What is this PR for?
This PR fixes a bug of showing checkbox for "Connect to existing process" 
on interpreter menu.


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


### How should this be tested?
Go to interpreter menu and check if checkbox for "Connect to existing 
process" exists.

### Screenshots (if appropriate)
- before

![image](https://cloud.githubusercontent.com/assets/3348133/17141824/aa316672-5388-11e6-952d-50a3769a01a5.png)

- after

![image](https://cloud.githubusercontent.com/assets/3348133/17141840/babfad8c-5388-11e6-9209-e37e252e95b4.png)


### 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/astroshim/zeppelin bugfix/showCheckbox

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

https://github.com/apache/zeppelin/pull/1229.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 #1229


commit bbc3283c7cf21fa7340887ddc3bf7ba7b5b8f689
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-26T14:24:41Z

show checkbox on interpreter menu.

commit 47ce719eec4076d02eb2895d87f69b2fbc6eaaa5
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-29T03:15:34Z

remove height style on the checkbox




---
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 pull request #1229: [BugFix] Show checkbox for "Connect to existing...

2016-08-01 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 #1229: [BugFix] Show checkbox for "Connect to existing proces...

2016-08-01 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1229
  
re-trigger CI


---
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 pull request #1229: [BugFix] Show checkbox for "Connect to existing...

2016-08-01 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[BugFix] Show checkbox for "Connect to existing process" on interpreter 
menu.

### What is this PR for?
This PR fixes a bug of showing checkbox for "Connect to existing process" 
on interpreter menu.


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


### How should this be tested?
Go to interpreter menu and check if checkbox for "Connect to existing 
process" exists.

### Screenshots (if appropriate)
- before

![image](https://cloud.githubusercontent.com/assets/3348133/17141824/aa316672-5388-11e6-952d-50a3769a01a5.png)

- after

![image](https://cloud.githubusercontent.com/assets/3348133/17141840/babfad8c-5388-11e6-9209-e37e252e95b4.png)


### 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/astroshim/zeppelin bugfix/showCheckbox

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

https://github.com/apache/zeppelin/pull/1229.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 #1229


commit bbc3283c7cf21fa7340887ddc3bf7ba7b5b8f689
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-26T14:24:41Z

show checkbox on interpreter menu.

commit 47ce719eec4076d02eb2895d87f69b2fbc6eaaa5
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-29T03:15:34Z

remove height style on the checkbox




---
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 pull request #1229: [BugFix] Show checkbox for "Connect to existing...

2016-07-26 Thread astroshim

GitHub user astroshim opened a pull request:

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

[BugFix] Show checkbox for "Connect to existing process" on interpreter 
menu.

### What is this PR for?
This PR fixes a bug of showing checkbox for "Connect to existing process" 
on interpreter menu.


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


### How should this be tested?
Go to interpreter menu and check if checkbox for "Connect to existing 
process" exists.

### Screenshots (if appropriate)
- before

![image](https://cloud.githubusercontent.com/assets/3348133/17141824/aa316672-5388-11e6-952d-50a3769a01a5.png)

- after

![image](https://cloud.githubusercontent.com/assets/3348133/17141840/babfad8c-5388-11e6-9209-e37e252e95b4.png)


### 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/astroshim/zeppelin bugfix/showCheckbox

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

https://github.com/apache/zeppelin/pull/1229.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 #1229


commit bbc3283c7cf21fa7340887ddc3bf7ba7b5b8f689
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-26T14:24:41Z

show checkbox on interpreter menu.




---
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 #1248: [ZEPPELIN-1069]Ignore implicit interpreter when user e...

2016-08-02 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1248
  
tested. LGTM


---
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 #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-11 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1193
  
@bzz I added log and tested. Thank you for taking care of this.


---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-11 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1318
  
Spark2.0 & hadoop2.3 is working well.

![image](https://cloud.githubusercontent.com/assets/3348133/17588549/d611d7ca-600a-11e6-87f6-4a692fbae273.png)



---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-11 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1318
  
Do you know what version of spark is popular? I will test 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.
---

[GitHub] zeppelin issue #1257: [Zeppelin-945] Interpreter authorization

2016-08-03 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1257
  
@AhyoungRyu I fixed you pointed out and I'll create the document for this 
separated PR. 
Thanks.


---
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 #1257: [Zeppelin-945] Interpreter authorization

2016-08-03 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1257
  
@AhyoungRyu Let me check. Thanks.


---
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 #1257: [Zeppelin-945] Interpreter authorization

2016-08-04 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1257
  
@prabhjyotsingh Great. Thank you for giving me the guidance.


---
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 #1257: [Zeppelin-945] Interpreter authorization

2016-08-04 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1257
  
Thank you for detail review @bzz and @felizbear 
I'll fix the codes.


---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-12 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1318
  
Then what about supporting latest verison(Spark2.0 & hadoop2.7)?


---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-12 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1318
  
Spark2.0 & hadoop2.7 is working well.
- YARN

![image](https://cloud.githubusercontent.com/assets/3348133/17622873/fc1b827c-60d7-11e6-80ec-e8e9890a5c99.png)

- hdfs 

![image](https://cloud.githubusercontent.com/assets/3348133/17622501/bdd23c1a-60d5-11e6-9aa2-a835e5fcecb2.png)

as you can see test `spark-submit` job was successful but zeppelin job 
doesn't work properly.
I'll take a look at problem later.



---
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 #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-08-12 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1193
  
@bzz I rebased. Thanks.


---
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 #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for running z...

2016-08-10 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1318
  
@AhyoungRyu I'll update. Thank you for giving me the informations.


---
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 pull request #1318: [ZEPPELIN-1280][Spark on Yarn] Documents for ru...

2016-08-10 Thread astroshim

GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1280][Spark on Yarn] Documents for running zeppelin on production 
environments.

### What is this PR for?
This PR is for the documentation of running zeppelin on production 
environments especially spark on yarn.
Related issue is https://github.com/apache/zeppelin/pull/1227 and I got a 
lot of hints from https://github.com/sequenceiq/hadoop-docker.
Tested on ubuntu.

### What type of PR is it?
Documentation

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


### 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/astroshim/zeppelin ZEPPELIN-1280

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

https://github.com/apache/zeppelin/pull/1318.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 #1318


commit 633c930af7c1561b3ec91687b20d1af0a576ba2f
Author: astroshim <hss...@nflabs.com>
Date:   2016-08-10T23:06:17Z

running zeppelin on yarn




---
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 #1171: [ZEPPELIN-1150] Table contents disappear 2nd run onwar...

2016-07-12 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1171
  
Build error has gone and working good!


---
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 #1176: [Zeppelin 1165 : improve] Paragraph dynamic relationsh...

2016-07-13 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1176
  
Great feature! 
Can you add documents for this or please explain more detail to review?


---
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 #1117: [ZEPPELIN-1086] Auto Completion for Interpreter.

2016-07-14 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1117
  
@bzz I totally agree with you so I did try to remove things you mentioned 
but it's not easy.
Let me try one more. Thanks!


---
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 #1117: [ZEPPELIN-1086] Auto Completion for Interpreter.

2016-07-15 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1117
  
@bzz Thank you kindly for telling like that. 
What do you think about tagging meta value than removing keywords like 
following image? 

![image](https://cloud.githubusercontent.com/assets/3348133/16870311/eb05d992-4abb-11e6-8172-99c90196a5c9.png)

I think it looks more nicer than removing keywords.
If it is okay, let me make it at the another 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.
---

[GitHub] zeppelin issue #1191: [ZEPPELIN-1186] Prevent NPE when unbind interpreter

2016-07-17 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1191
  
LGTM


---
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 #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-07-19 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1193
  
please review this. :)


---
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 pull request #1193: [ZEPPELIN-1192] Block pyspark paragraph hang.

2016-07-15 Thread astroshim

GitHub user astroshim opened a pull request:

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

[ZEPPELIN-1192] Block pyspark paragraph hang.

### What is this PR for?
This PR block pyspark paragraph hang.


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


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


### How should this be tested?
1. build zeppelin without pyspark (just mvn clean package -DskipTests)
2. open note and set paragraph interpreter as "spark.pyspark" 
3. hit "Ctrl+." for auto completion.
4. try run paragraph.
- please refer to the screenshot.

### Screenshots (if appropriate)
- before

![b](https://cloud.githubusercontent.com/assets/3348133/16881827/ee30e248-4af6-11e6-9409-7e7b9f622121.gif)

- after

![a](https://cloud.githubusercontent.com/assets/3348133/16881840/f4d6d2ec-4af6-11e6-89b3-1e4e2806a742.gif)


### 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/astroshim/zeppelin ZEPPELIN-1192

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

https://github.com/apache/zeppelin/pull/1193.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 #1193


commit 288eca7ddb8bffd271221ad21805f07d2181f8b5
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-15T16:37:04Z

block infinite loop.




---
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 #1196: [ZEPPELIN-1195] Unbind interpreter error.

2016-07-16 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1196
  
Oh sorry.


---
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 pull request #1185: Redirect home when requested a wrong notename p...

2016-07-14 Thread astroshim

GitHub user astroshim reopened a pull request:

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

Redirect home when requested a wrong notename path.

### What is this PR for?
This PR for redirecting to zeppelin home when requested wrong path.


### What type of PR is it?
Improvement


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


### How should this be tested?
put wrong path to your browser like screenshot.


### Screenshots (if appropriate)
- before

![b1](https://cloud.githubusercontent.com/assets/3348133/16835556/3b744224-49f4-11e6-84bf-6a22cc729a30.gif)

- after

![a](https://cloud.githubusercontent.com/assets/3348133/16835627/8cc590e2-49f4-11e6-878f-745ee026ef8e.gif)




### 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/astroshim/zeppelin feat/redirectHome

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

https://github.com/apache/zeppelin/pull/1185.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 #1185


commit 8db7017742f9b579c5fe2c09ce827d1c258d5ecd
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-14T09:41:04Z

redirect home when wrong path

commit c937b90bccdc2f13877993684e637c9078859b50
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-15T03:44:20Z

fix location.path

commit eae453f5c02a990c8c92418c5b30ebffe6c7df7d
Author: astroshim <hss...@nflabs.com>
Date:   2016-07-15T03:45:55Z

fix path




---
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 #1090: [MINOR] Blocking the minus cursor value.

2016-06-27 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1090
  
Thank you for your review! @bzz 


---
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 #1030: [ZEPPELIN-1023] Add more credential apis.

2016-06-27 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1030
  
@bzz okay I will. Thank you!


---
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 #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1097
  
It seems that the CI error is not related this PR. 
```
Tests in error: 
  
ZeppelinIT.testSparkInterpreterDependencyLoading:217->AbstractZeppelinIT.createNewNote:119
 ÃÂ» Timeout
```


---
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 #1100: [ZEPPELIN-1054] Improve "Credentials" UI

2016-06-28 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1100
  
Really great feature! Thanks for making this.


---
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 #1099: ZEPPELIN-1071 ] Ace-editor hidden auto-complete additi...

2016-06-28 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1099
  
LGTM!


---
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 pull request #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 pull request #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-29 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[ZEPPELIN-1078] Does not show the text result in the paragraph sometimes.

### What is this PR for?
This PR fixes the problem that does not show the text result in the 
paragraph sometimes.


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


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


### How should this be tested?
Try run python code constantly like screenshot.

### Screenshots (if appropriate)
  - before

![before](https://cloud.githubusercontent.com/assets/3348133/16436829/214a2b4e-3ddc-11e6-9af2-2ee1d7e2cf96.gif)


### 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/astroshim/zeppelin ZEPPELIN-1078

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

https://github.com/apache/zeppelin/pull/1104.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 #1104


commit 441357c042ed557c63f18bb15b2833293431bd06
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-29T00:28:41Z

remove resultRefreshed value.




---
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 pull request #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread astroshim

GitHub user astroshim reopened a pull request:

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

[Bugfix] init RepoSetting values

### What is this PR for?
This PR fixes initialize value bug of ```Add New Repository```.


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


### How should this be tested?
1. launch the ```Add New Repository``` modal window.
2. put URL value 
3. hit Cancel button
4. launch the ```Add New Repository``` modal window again.


### Screenshots (if appropriate)
  - before

![bb](https://cloud.githubusercontent.com/assets/3348133/16388909/5fbddd00-3cd6-11e6-9e46-1afbb8a37356.gif)


  - after

![after](https://cloud.githubusercontent.com/assets/3348133/16388911/6294390c-3cd6-11e6-92ae-01a8730e3c25.gif)


### 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/astroshim/zeppelin bugfix/initRepositoryValues

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

https://github.com/apache/zeppelin/pull/1097.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 #1097


commit f5a08d544d91082c807e5a3c0b5cc93b09698708
Author: astroshim <hss...@nflabs.com>
Date:   2016-06-27T17:08:47Z

init newRepoSetting values




---
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 pull request #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread astroshim

Github user astroshim closed the pull request at:

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


---
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 #1097: [Bugfix] init RepoSetting values

2016-06-29 Thread astroshim

Github user astroshim commented on the issue:

https://github.com/apache/zeppelin/pull/1097
  
CI has passed. Thanks.


---
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 pull request #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread astroshim

Github user astroshim closed the pull request at:

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


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

1 2 3 4 5 >

1 - 100 of 435 matches

Mail list logo