[GitHub] zeppelin pull request #1097: [Bugfix] init RepoSetting values
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
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: astroshimDate: 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...
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
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...
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...
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
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
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
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...
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
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...
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 LeeDate: 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...
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...
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...
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: astroshimDate: 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...
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
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...
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
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...
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: astroshimDate: 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.
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
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...
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
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...
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 LeeDate: 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
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 ...
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
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
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
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
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
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.
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: astroshimDate: 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.
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
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...
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
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...
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
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
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.
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...
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...
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 SinghDate: 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
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
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
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: AhyoungRyuDate: 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...
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: CloverHeartsDate: 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
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
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