[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 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 
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 #1102: [ZEPPELIN-1075] Merge NoteInterpreterLoader into Inter...

2016-06-28 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1102
  
CI becomes green. I'll fix style of some 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 #1097: [Bugfix] init RepoSetting values

2016-06-28 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1097
  
It's flaky test but we need to ensure all the test passed. Could you please 
do it once again?


---
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 prabhjyotsingh
Github user prabhjyotsingh 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 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 #1056: [TEST] Selenium Tests 01

2016-06-28 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


---
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 #1068: [TEST] CI -BUILD TEST - TASK 1

2016-06-28 Thread cloverhearts
Github user cloverhearts closed the pull request at:

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


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


[jira] [Created] (ZEPPELIN-1080) Remove paragraph does not work

2016-06-28 Thread Yesha Vora (JIRA)
Yesha Vora created ZEPPELIN-1080:


 Summary: Remove paragraph does not work
 Key: ZEPPELIN-1080
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1080
 Project: Zeppelin
  Issue Type: Bug
Reporter: Yesha Vora


Steps to reproduce:
* Create a notebook.
* create paragraphs in a notebook
* Click on setting icon for the paragraph and select remove.

The paragraph should be removed.
Actual behavior : paragraph is not actually removed.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin pull request #1102: [ZEPPELIN-1075] Merge NoteInterpreterLoader int...

2016-06-28 Thread jongyoul
Github user jongyoul closed the pull request at:

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


---
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
  
I re-triggered but other error occurred.
```
Tests run: 5, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 0.579 sec 
<<< FAILURE! - in org.apache.zeppelin.python.PythonInterpreterTest
testClose(org.apache.zeppelin.python.PythonInterpreterTest)  Time elapsed: 
0.429 sec  <<< FAILURE!
java.lang.AssertionError: null
at org.junit.Assert.fail(Assert.java:86)
at org.junit.Assert.assertTrue(Assert.java:41)
at org.junit.Assert.assertFalse(Assert.java:64)
at org.junit.Assert.assertFalse(Assert.java:74)
at 
org.apache.zeppelin.python.PythonInterpreterTest.testClose(PythonInterpreterTest.java:141)


Results :

Failed tests: 
  PythonInterpreterTest.testClose:141 null

```


---
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 #1102: [ZEPPELIN-1075] Merge NoteInterpreterLoader int...

2016-06-28 Thread jongyoul
GitHub user jongyoul reopened a pull request:

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

[ZEPPELIN-1075] Merge NoteInterpreterLoader into InterpreterFactory

### What is this PR for?
Removing redundant codes between `NoteInterpreterLoader` and 
`InterpreterFactory`, reducing the cost to add new features, and making 
refactoring on `InterpreterFactory` easy

### What type of PR is it?
[Refactoring]

### Todos
* [x] - Remove `NoteInterpreterLoader` and move the functionality into 
`InterpreterFactory`

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

### How should this be tested?
Must pass all tests

### Screenshots (if appropriate)

### 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/jongyoul/zeppelin ZEPPELIN-1075

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

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


commit 9c3c82db5f2599cd1fbd9ef3faa8abf71fd349ac
Author: Jongyoul Lee 
Date:   2016-06-28T16:37:22Z

Duplicated all method in NoteInterpreterLoader to InterpreterFactory

commit a2115d37ab37064f3469a37f8a12c1beb273707d
Author: Jongyoul Lee 
Date:   2016-06-28T17:00:36Z

Removed NoteInterpreterLoader
Fixed some tests

commit 9a13a752fae16a8ace6f47ad64a0edb9b627a1e6
Author: Jongyoul Lee 
Date:   2016-06-29T01:20:09Z

Fixed style




---
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 #1102: [ZEPPELIN-1075] Merge NoteInterpreterLoader int...

2016-06-28 Thread jongyoul
Github user jongyoul closed the pull request at:

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


---
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 #1103: [ZEPPELIN-1076] Set hbase.client.retries.number for JD...

2016-06-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1103
  
Yes, `phoenix.hbase.client.retries.number`, and have tested with and 
without this string.


---
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-28 Thread astroshim
GitHub user astroshim reopened 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 
Date:   2016-06-22T05:04:26Z

disallow the blank values of Credential on the front.

commit 63bc3852995717be20d5a144918c24a2376bce2d
Author: astroshim 
Date:   2016-06-22T11:38:45Z

allow blank password.




---
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-28 Thread astroshim
Github user astroshim closed the pull request at:

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


---
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 #908: ZEPPELIN-598 ] Dynamic loading for Interpreter and API

2016-06-28 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/908
  
@ggdupont 
Sorry. The first answer seems much too late.
OK, it seems to do the modification.
In a somehow given log, store, see a problem I think.
You can modify the repository maven setting.xml change by force.
(For example, you can replace the repo1 repository mirror set).
I'll check for more details.
Thank you for your feedback.


---
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 #1103: [ZEPPELIN-1076] Set hbase.client.retries.number for JD...

2016-06-28 Thread jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1103
  
LGTM. That property `hbase.client.retries.number` will be passed to phoenix 
jdbc, right?


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


[jira] [Created] (ZEPPELIN-1079) Zeppelin server fails with Null pointer exception

2016-06-28 Thread Yesha Vora (JIRA)
Yesha Vora created ZEPPELIN-1079:


 Summary: Zeppelin server fails with Null pointer exception
 Key: ZEPPELIN-1079
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1079
 Project: Zeppelin
  Issue Type: Bug
  Components: zeppelin-server
Reporter: Yesha Vora
Priority: Critical


Zeppelin-server gets Null Pointer Exception. 
This cluster has Spark 1.6 + Livy and Zeppelin installed. 

{code}
WARN [2016-06-29 00:03:38,692] ({qtp471108564-16} 
SecurityRestApi.java[ticket]:84) - 
{"status":"OK","message":"","body":{"principal":"anonymous","ticket":"anonymous","roles":"[]"}}
 INFO [2016-06-29 00:03:38,716] ({qtp471108564-17} 
NotebookServer.java[onOpen]:111) - New connection from xx.xx.xx.xx : 61088
 INFO [2016-06-29 00:03:38,788] ({qtp471108564-18} 
NotebookServer.java[onOpen]:111) - New connection from xx.xx.xx.xx : 61105
 INFO [2016-06-29 00:03:38,945] ({qtp471108564-17} 
NotebookServer.java[sendNote]:475) - New operation from xx.xx.xx.xx : 61105 : 
anonymous : GET_NOTE : 2BQMC8AUD
 INFO [2016-06-29 00:03:39,003] ({qtp471108564-18} 
NotebookServer.java[sendNote]:475) - New operation from xx.xx.xx.xx : 61088 : 
anonymous : GET_NOTE : 2BQMC8AUD
 INFO [2016-06-29 00:03:43,450] ({pool-1-thread-1} 
SchedulerFactory.java[jobStarted]:131) - Job paragraph_1467153201430_1258004256 
started by scheduler 
org.apache.zeppelin.interpreter.remote.RemoteInterpretershared_session671256830
 INFO [2016-06-29 00:03:43,453] ({pool-1-thread-1} Paragraph.java[jobRun]:252) 
- run paragraph 20160628-223321_1848419437 using null 
org.apache.zeppelin.interpreter.LazyOpenInterpreter@7258adc3
 INFO [2016-06-29 00:03:43,454] ({pool-1-thread-1} 
RemoteInterpreter.java[init]:170) - Create remote interpreter 
org.apache.zeppelin.spark.SparkInterpreter
 WARN [2016-06-29 00:03:43,454] ({pool-1-thread-1} 
RemoteInterpreterProcess.java[releaseClient]:198) - exception occurred during 
releasing thrift client
java.lang.NullPointerException
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreterProcess.releaseClient(RemoteInterpreterProcess.java:196)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.init(RemoteInterpreter.java:186)
at 
org.apache.zeppelin.interpreter.remote.RemoteInterpreter.getFormType(RemoteInterpreter.java:328)
at 
org.apache.zeppelin.interpreter.LazyOpenInterpreter.getFormType(LazyOpenInterpreter.java:105)
at org.apache.zeppelin.notebook.Paragraph.jobRun(Paragraph.java:260)
at org.apache.zeppelin.scheduler.Job.run(Job.java:176)
at 
org.apache.zeppelin.scheduler.RemoteScheduler$JobRunner.run(RemoteScheduler.java:328)
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){code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin pull request #1104: [ZEPPELIN-1078] Does not show the text result i...

2016-06-28 Thread astroshim
GitHub user astroshim opened 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 
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.
---


[jira] [Created] (ZEPPELIN-1078) Does not show the text result in the paragraph sometimes.

2016-06-28 Thread Shim HyungSung (JIRA)
Shim HyungSung created ZEPPELIN-1078:


 Summary: Does not show the text result in the paragraph sometimes.
 Key: ZEPPELIN-1078
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1078
 Project: Zeppelin
  Issue Type: Bug
Reporter: Shim HyungSung


Does not show the text result in the paragraph sometimes when I ran python code.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[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 #980: [ZEPPELIN-871] [WIP] spark 2.0 interpreter on scala 2.1...

2016-06-28 Thread rnirmal
Github user rnirmal commented on the issue:

https://github.com/apache/zeppelin/pull/980
  
@echarles @Leemoonsoo do you know where the direction lies for Spark 2.0 
with scala 2.11 support.. i.e as separate items or continued effort on this PR? 
Since Spark 2.0 is going to be release in the next week or two, would be great 
to see this made available on Zeppelin as a quick follow on.

I can help with any testing if needed, not as familiar with Zeppelin code 
yet to help with reviews. 


---
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 jongyoul
Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1097
  
Could you please re-trigger 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 pull request #1102: [ZEPPELIN-1075] Merge NoteInterpreterLoader int...

2016-06-28 Thread jongyoul
GitHub user jongyoul opened a pull request:

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

[ZEPPELIN-1075] Merge NoteInterpreterLoader into InterpreterFactory

### What is this PR for?
Removing redundant codes between `NoteInterpreterLoader` and 
`InterpreterFactory`, reducing the cost to add new features, and making 
refactoring on `InterpreterFactory` easy

### What type of PR is it?
[Refactoring]

### Todos
* [x] - Remove `NoteInterpreterLoader` and move the functionality into 
`InterpreterFactory`

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

### How should this be tested?
Must pass all tests

### Screenshots (if appropriate)

### 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/jongyoul/zeppelin ZEPPELIN-1075

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

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


commit 9c3c82db5f2599cd1fbd9ef3faa8abf71fd349ac
Author: Jongyoul Lee 
Date:   2016-06-28T16:37:22Z

Duplicated all method in NoteInterpreterLoader to InterpreterFactory

commit a2115d37ab37064f3469a37f8a12c1beb273707d
Author: Jongyoul Lee 
Date:   2016-06-28T17:00:36Z

Removed NoteInterpreterLoader
Fixed some tests




---
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 #1080: [ZEPPELIN-1060] validate user before saving

2016-06-28 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1086: [Zeppelin 1042] Extra space is present as part ...

2016-06-28 Thread asfgit
Github user asfgit closed the pull request at:

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


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


[jira] [Created] (ZEPPELIN-1075) Move the functionality of NoteInterpreterLoader into InterpreterFactory

2016-06-28 Thread Jongyoul Lee (JIRA)
Jongyoul Lee created ZEPPELIN-1075:
--

 Summary: Move the functionality of NoteInterpreterLoader into 
InterpreterFactory
 Key: ZEPPELIN-1075
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1075
 Project: Zeppelin
  Issue Type: Task
  Components: zeppelin-zengine
Reporter: Jongyoul Lee


{{NoteInterpreterLoader}} stores {{noteId}} only, and is used by {{Note}} and 
{{Paragraph}}. Those classes, however, already have a {{noteId}} by themselves. 
Furthermore {{NoteInterpreterLoader}}'s method are same or similar mapper of 
{{InterpreterFactory}}. Thus there's no reason to keep 
{{NoteInterpreterLoader}} and it would be better move those functions into 
{{InterpreterFactory}}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1069: [ZEPPELIN-281] Windows7 build2

2016-06-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1069
  
Thanks for the investigation.
CI failure is irrelevant while this PR changes `README.md` only.

LGTM and merge if there're no more discussions



---
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-28 Thread Leemoonsoo
Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1025
  
IMO, 'Job' menu item can stay next to 'Notebook', instead of going into 
dropdown menu.
Because, "Interpreter', "Credential", "Configuration" menus are related 
configuring Zeppelin, and "Notebook", "Job" are related to using Zeppelin.


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

2016-06-28 Thread astroshim
GitHub user astroshim reopened 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 -XDELETE -H "Content-Type: application/json" 
"http://localhost:8080/api/credential/e1;
```

  - to remove all credential information.
```
curl -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? no



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 
Date:   2016-06-16T15:05:42Z

add more credential apis.

commit 9b2c1c98db582b88e93d66d7ccbe3427b4add607
Author: astroshim 
Date:   2016-06-18T13:40:15Z

add checking null and blank values in the putCredentials method.

commit 26433f2945104e17cdbc70de13bfa3f0ca5395e8
Author: astroshim 
Date:   2016-06-23T05:34:01Z

change using syncronizedMap.

commit f54653282b26cb087ce52e1942ed513999d35c8a
Author: astroshim 
Date:   2016-06-24T15:23:41Z

add rest-credential document

commit 80d8a30ee2912b4952a305b88d9a12617a3bd325
Author: AhyoungRyu 
Date:   2016-06-24T16:18:08Z

Add rest-credential.md to index.md & dropdown menu

commit 7b2a7c5709dd31ecfb3180f0d8a236325d5e8b4f
Author: HyungSung 
Date:   2016-06-24T16:47:35Z

Merge pull request #7 from AhyoungRyu/ZEPPELIN-1023-docs

[ZEPPELIN-1023] Add rest-credential.md to index.md & dropdown menu

commit d0c2bac7a42c3a0b4d61a1e0dd2901870925a1c0
Author: astroshim 
Date:   2016-06-24T21:30:10Z

changed syncronizedmap to concurrenthashmap

commit 0f503fdf4c46b1b3de8f4a42d982830b3ccca3a7
Author: astroshim 
Date:   2016-06-27T11:42:34Z

to fix ci

commit 06099313cda64de3850652b4b5d5d4b1d1647d02
Author: astroshim 
Date:   2016-06-27T13:08:05Z

update credentialsMap

commit a58047624b0c1171a3586edc8eeb00e7f12985ff
Author: astroshim 
Date:   2016-06-27T13:38:43Z

erase comment code

commit 1b94ae12b00cba565f879cc44c76758432b66331
Author: astroshim 
Date:   2016-06-27T13:38:47Z

Merge branch 'master' into ZEPPELIN-1023

rebase




---
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-28 Thread astroshim
Github user astroshim closed the pull request at:

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


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


[jira] [Created] (ZEPPELIN-1074) zeppelin not working with https

2016-06-28 Thread shruti (JIRA)
shruti created ZEPPELIN-1074:


 Summary: zeppelin not working with https
 Key: ZEPPELIN-1074
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1074
 Project: Zeppelin
  Issue Type: Bug
  Components: zeppelin-server
Affects Versions: 0.5.6
 Environment: aws linux 
Reporter: shruti
Priority: Blocker
 Fix For: 0.5.6


http://carminedimascio.com/2015/06/running-zeppelin-with-ssl/
i followed the steps given the above link, but still not able to launch the  
notebook with https/.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1064: BugFix-blocking of blank values insertion on the Crede...

2016-06-28 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1064
  
Thank you!
Looks good to me, mering if there is no further discussion


---
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 #1094: ZEPPELIN-1063: fix flaky python interpreter test

2016-06-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1094
  
Yes, I've seen this, this happens when spark takes longer that 60sec to 
start up. 
Will open up a new jira and fix this as well.


---
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 #1094: ZEPPELIN-1063: fix flaky python interpreter tes...

2016-06-28 Thread asfgit
Github user asfgit closed the pull request at:

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


---
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 #1094: ZEPPELIN-1063: fix flaky python interpreter test

2016-06-28 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1094
  
CI failure is not related and handled under 
[ZEPPELIN-1073](https://issues.apache.org/jira/browse/ZEPPELIN-1073)
Merging if there is no further disucssion


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


[jira] [Created] (ZEPPELIN-1073) Falky Test - ParagraphActionsIT.testClearOutputButton

2016-06-28 Thread Alexander Bezzubov (JIRA)
Alexander Bezzubov created ZEPPELIN-1073:


 Summary: Falky Test - ParagraphActionsIT.testClearOutputButton
 Key: ZEPPELIN-1073
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1073
 Project: Zeppelin
  Issue Type: Bug
Reporter: Alexander Bezzubov


CI fails i.e on

 - [#1094|https://github.com/apache/zeppelin/pull/1094], 
[logs|https://s3.amazonaws.com/archive.travis-ci.org/jobs/140563773/log.txt]

{code}
Driver info: driver.version: unknown
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
at 
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:57)
at 
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
at java.lang.reflect.Constructor.newInstance(Constructor.java:526)
at 
org.openqa.selenium.remote.ErrorHandler.createThrowable(ErrorHandler.java:206)
at 
org.openqa.selenium.remote.ErrorHandler.throwIfResponseFailed(ErrorHandler.java:158)
at 
org.openqa.selenium.remote.RemoteWebDriver.execute(RemoteWebDriver.java:647)
at 
org.openqa.selenium.remote.RemoteWebDriver.findElement(RemoteWebDriver.java:353)
at 
org.openqa.selenium.remote.RemoteWebDriver.findElementByXPath(RemoteWebDriver.java:490)
at org.openqa.selenium.By$ByXPath.findElement(By.java:361)
at 
org.openqa.selenium.remote.RemoteWebDriver.findElement(RemoteWebDriver.java:345)
at 
org.apache.zeppelin.AbstractZeppelinIT$1.apply(AbstractZeppelinIT.java:98)
at 
org.apache.zeppelin.AbstractZeppelinIT$1.apply(AbstractZeppelinIT.java:96)
at org.openqa.selenium.support.ui.FluentWait.until(FluentWait.java:238)
at 
org.apache.zeppelin.AbstractZeppelinIT.pollingWait(AbstractZeppelinIT.java:96)
at 
org.apache.zeppelin.AbstractZeppelinIT.waitForParagraph(AbstractZeppelinIT.java:70)
at 
org.apache.zeppelin.integration.ParagraphActionsIT.testClearOutputButton(ParagraphActionsIT.java:285)
Caused by: org.openqa.selenium.NoSuchElementException: Unable to locate 
element: 
{"method":"xpath","selector":"(//div[@ng-controller=\"ParagraphCtrl\"])[1]//div[contains(@class,
 'control')]//span[1][contains(.,'FINISHED')]"}


Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 143.455 sec - 
in org.apache.zeppelin.integration.ZeppelinIT

Results :

Tests in error: 
  
ParagraphActionsIT.testClearOutputButton:285->AbstractZeppelinIT.waitForParagraph:70->AbstractZeppelinIT.pollingWait:96
 » Timeout

Tests run: 15, Failures: 0, Errors: 1, Skipped: 0

{code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1090: [MINOR] Blocking the minus cursor value.

2016-06-28 Thread bzz
Github user bzz commented on the issue:

https://github.com/apache/zeppelin/pull/1090
  
Merging if there is no further discussion


---
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 #1088: [ZEPPELIN-1062] Get original InterpreterSetting when t...

2016-06-28 Thread prabhjyotsingh
Github user prabhjyotsingh commented on the issue:

https://github.com/apache/zeppelin/pull/1088
  
Also what happens when user clicks on close ("x") button, instead of "OK" 
or "Close" ?


---
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 #1101: [ZEPPELIN-1072] Saving Interpreter Setting dose...

2016-06-28 Thread prabhjyotsingh
GitHub user prabhjyotsingh opened a pull request:

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

[ZEPPELIN-1072] Saving Interpreter Setting dosen't notify when its finished

### What is this PR for?
When user tries to save Interpreter Setting, it does not notify user when 
its done.
In cases where saving setting includes adding dependencies, it can take 
longer than 10 seconds, depending on number of dependencies added.

### What type of PR is it?
[Improvement]

### Todos
* [x] - add fa-spin with OK button

### What is the Jira issue?
* [ZEPPELIN-1072](https://issues.apache.org/jira/browse/ZEPPELIN-1072)

### How should this be tested?
Try loading a dependency in any of the interpreter, you will notice "OK" 
button is replaced by loading icon.

### Screenshots (if appropriate)

![zeppelin-1072](https://cloud.githubusercontent.com/assets/674497/16409449/3df5fb34-3d3b-11e6-8d3e-457022e7269e.gif)


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



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

$ git pull https://github.com/prabhjyotsingh/zeppelin ZEPPELIN-1072

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

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


commit 8d4387c026e402e83804faab5e4d569d33cddf8e
Author: Prabhjyot Singh 
Date:   2016-06-28T08:27:17Z

add fa-spin with OK button




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


[jira] [Created] (ZEPPELIN-1072) Saving Interpreter Setting dosen't notify when its finished

2016-06-28 Thread Prabhjyot Singh (JIRA)
Prabhjyot Singh created ZEPPELIN-1072:
-

 Summary: Saving Interpreter Setting dosen't notify when its 
finished
 Key: ZEPPELIN-1072
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1072
 Project: Zeppelin
  Issue Type: Bug
Reporter: Prabhjyot Singh
Assignee: Prabhjyot Singh
Priority: Minor


When user tries to save Interpreter Setting, it does not notify user when its 
done. 

In cases where saving setting includes adding dependencies, it can take longer 
than 10 seconds, depending on number of dependencies added.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[GitHub] zeppelin issue #1025: ZEPPELIN-964 ] Jobmanagement - (2) basic front-end

2016-06-28 Thread cloverhearts
Github user cloverhearts commented on the issue:

https://github.com/apache/zeppelin/pull/1025
  
merge from master branch and replace job menu location.
https://cloud.githubusercontent.com/assets/10525473/16408386/7601cf18-3d53-11e6-99d3-2b6f34691af9.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 pull request #1100: [ZEPPELIN-1054] Improve "Credentials" UI

2016-06-28 Thread AhyoungRyu
GitHub user AhyoungRyu opened a pull request:

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

[ZEPPELIN-1054] Improve "Credentials" UI 

### What is this PR for?
Currently, users can add new their credential info for data source 
authentication in Zeppelin "Credentials" menu. Even though it was saved 
successfully, they can't see the whole list of credentials in Zeppelin UI. 
This PR enables to `get` all credential list, `edit` and `remove` via UI. 

*NOTE : Since this patch was implemented based on #1030 API, should be 
tested after #1030 merged.*

### What type of PR is it?
Improvement & Documentation

### Todos
* [x] - rename `interpreter_authorization.md` -> 
`datasource_authorization.md`
* [x] - remove `Interpreter Authorization` section (since we don't have 
this feature yet : 
[ZEPPELIN-945](https://issues.apache.org/jira/browse/ZEPPELIN-945))
* [ ] - rebase after #1030 & #1064 merged

### What is the Jira issue?
[ZEPPELIN-1054](https://issues.apache.org/jira/browse/ZEPPELIN-1054)

### How should this be tested?
1. Apply this patch and build `zeppelin-web` as described in 
[here](https://github.com/apache/zeppelin/tree/master/zeppelin-web#configured-environment).
2. Go to `Credentials` menu.
3. Add new credentials -> you can see the credential info in the credential 
list table. 
4. You can edit & delete them. -> Compare with `conf/credentials.json`

### Screenshots (if appropriate)
- Before 
https://cloud.githubusercontent.com/assets/10060731/16407604/69b0c4d8-3cc9-11e6-8284-9abe2969cdc1.png;>

- After 

![add_credential](https://cloud.githubusercontent.com/assets/10060731/16407612/6e266068-3cc9-11e6-955d-572fd4eca277.gif)
If there is no credential
https://cloud.githubusercontent.com/assets/10060731/16407620/7838995e-3cc9-11e6-90ba-1bd0173a1b49.png;>

- `datasource_authorization.md`
https://cloud.githubusercontent.com/assets/10060731/16407680/d176ddf0-3cc9-11e6-9070-178400ad7c86.png;>
https://cloud.githubusercontent.com/assets/10060731/16407681/d4168d8a-3cc9-11e6-9319-018ae440cef8.png;>
https://cloud.githubusercontent.com/assets/10060731/16407682/d5f36a10-3cc9-11e6-8b9a-13b09960cda6.png;>
 
### 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/AhyoungRyu/zeppelin ZEPPELIN-1054

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

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


commit 3681e96d84e8fe9094c3149685e61b51e914eab0
Author: AhyoungRyu 
Date:   2016-06-27T17:35:02Z

Enable credential info to get list, edit and remove via UI

commit 50ea0ac2549b70bdd96bd292bc3be63a0f5b7060
Author: AhyoungRyu 
Date:   2016-06-27T17:35:25Z

Update angular-xeditable version

commit 00cc690b1a2fc17c35b7c76d703c8f9f500450b5
Author: AhyoungRyu 
Date:   2016-06-27T17:34:11Z

Add datasource_authorization.md to index & navi menu

commit 071f18f82b71719b2e88aafbe77734d17ec02701
Author: AhyoungRyu 
Date:   2016-06-28T07:14:33Z

Add datasource_authorization.md docs

commit f7c6a7d1832f3d7516e3388b2fdbe5e6a81bcc9c
Author: AhyoungRyu 
Date:   2016-06-28T07:56:33Z

Fix indentation

commit 32e5809a9d7c80cfcd83a6b2501f8dc50dbbf537
Author: AhyoungRyu 
Date:   2016-06-28T08:03:55Z

Delete interpreter_authorization.md




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

2016-06-28 Thread cloverhearts
GitHub user cloverhearts opened a pull request:

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

ZEPPELIN-1071 ] Ace-editor hidden auto-complete additional events.

### What is this PR for?
Does not hide popup for Ace editor-autocomplete
If the page move or click on an event occurs, the auto-completion should be 
hidden.

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

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

### How should this be tested?
1. Call the auto-complete in the code window (ctrl + shift + space)
2. Click anywhere without closing the pop-up.

### Screenshots (if appropriate)

 Before

![ace_ok](https://cloud.githubusercontent.com/assets/10525473/16407960/56d71fe6-3d51-11e6-8a39-5938388de8a4.gif)
 After

![ace_ok2](https://cloud.githubusercontent.com/assets/10525473/16407961/593ae7f4-3d51-11e6-8deb-6c7106b060c8.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/cloverhearts/zeppelin fix/acecomplete_hide

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

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


commit af8a16ef92ff271885e1c26cb55d56ca00cc38d0
Author: CloverHearts 
Date:   2016-06-28T07:50:04Z

Ace-editor hidden auto-complete additional events.




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


[jira] [Created] (ZEPPELIN-1071) does not hide popup for Ace Editor-Autocomplete

2016-06-28 Thread CloverHearts (JIRA)
CloverHearts created ZEPPELIN-1071:
--

 Summary: does not hide popup for Ace Editor-Autocomplete
 Key: ZEPPELIN-1071
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-1071
 Project: Zeppelin
  Issue Type: Bug
Reporter: CloverHearts
Assignee: CloverHearts


Does not hide popup for Ace editor-autocomplete
If the page move or click on an event occurs, the auto-completion should be 
hidden.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Design of solution for communication and concurrency of Ipfs/Bittorent

2016-06-28 Thread onkar shedge
The note to be downloaded from peers concurrently.So I read about Executor
service and I have used google-gauva for that, using callbacks to decide
what to do on completion of task. I have doubt on how the design should be
before proceeding forward.
I have posted my query on the blog[1] . Could you please go through it and
give feedback.

1]
http://gsoc2016onkar.blogspot.in/2016/06/communication-websocket-and-concurrency.html

Thanks,
Onkar