You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by meenakshisekar <gi...@git.apache.org> on 2016/10/21 13:34:30 UTC
[GitHub] zeppelin pull request #1548: Commented the properties in interpreter file an...
GitHub user meenakshisekar opened a pull request:
https://github.com/apache/zeppelin/pull/1548
Commented the properties in interpreter file and added setting file.
### What is this PR for?
This handles replacing the registration of interpreter with static block by the interpreter-setting.json file
### What type of PR is it?
[| Improvement |
### Todos
Sub-Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-917
### How should this be tested?
here shouldn't be any warning like below on starting the server
INFO [2016-09-29 00:25:46,247] ({main} LensInterpreter.java[<clinit>]:155) - Bootstrapping Cassandra Interpreter WARN [2016-09-29 00:25:46,250] ({main} Interpreter.java[register]:347) - Static initialization is deprecated for interpreter lens, You should change it to use interpreter-setting.json in your jar or interpreter/{interpreter}/interpreter-setting.json INFO [2016-09-29 00:25:46,250] ({main} InterpreterFactory.java[init]:204) - Inclass=org.apache.zeppelin.lens.LensInterpreter
And ensure that the lens related paragraphs run without any error
### 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/meenakshisekar/zeppelin ZEPPELIN-917
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1548.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 #1548
----
commit db2827f867199cdd5e3fe2bc05f614dd88e013f5
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T09:59:38Z
Commented the properties in interpreter file and added setting file.
commit f94c8a3d44d441174fff2e89708e9ec62c1f5d22
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T11:41:58Z
Commented the properties in interpreter file and added setting file.
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: Commented the properties in interpreter file and added...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1548
Can you change the title of this PR including jira issue number and title?
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar closed the pull request at:
https://github.com/apache/zeppelin/pull/1548
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
GitHub user meenakshisekar reopened a pull request:
https://github.com/apache/zeppelin/pull/1548
[ZEPPELIN-917] Apply new mechanism to LensInterpreter
### What is this PR for?
This handles replacing the registration of interpreter with static block by the interpreter-setting.json file
### What type of PR is it?
[| Improvement |
### Todos
Sub-Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-917
### How should this be tested?
here shouldn't be any warning like below on starting the server
INFO [2016-09-29 00:25:46,247]({main} LensInterpreter.java[<clinit>]:155) - Bootstrapping Cassandra Interpreter WARN [2016-09-29 00:25:46,250]({main} Interpreter.java[register]:347) - Static initialization is deprecated for interpreter lens, You should change it to use interpreter-setting.json in your jar or interpreter/{interpreter}/interpreter-setting.json INFO [2016-09-29 00:25:46,250]({main} InterpreterFactory.java[init]:204) - Inclass=org.apache.zeppelin.lens.LensInterpreter
And ensure that the lens related paragraphs run without any error
### 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/meenakshisekar/zeppelin ZEPPELIN-917
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1548.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 #1548
----
commit db2827f867199cdd5e3fe2bc05f614dd88e013f5
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T09:59:38Z
Commented the properties in interpreter file and added setting file.
commit f94c8a3d44d441174fff2e89708e9ec62c1f5d22
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T11:41:58Z
Commented the properties in interpreter file and added setting file.
commit ad3b83ca856c54727f1ffef324f187f69a276c83
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-24T11:57:46Z
Added indentation
commit 2cba9e74df90073bd03c2867a70c345e5346dc7e
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T07:12:54Z
Added the indentation similar to other interpreter setting file.
commit cc8bdc274f581e7725307e2a4706b4d8ce4e243c
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T07:12:54Z
Added the indentation similar to other interpreter setting file.
commit 269e8cb3c547d5f1ea932c571206c13931a462ca
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T08:02:10Z
Removed the tabs and added space.
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/zeppelin/pull/1548
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1548
@meenakshisekar Yes. Can you see the difference about the indentation between [your change](https://github.com/meenakshisekar/zeppelin/blob/ad3b83ca856c54727f1ffef324f187f69a276c83/lens/src/main/resources/interpreter-setting.json) and [spark interpreter's `interpreter-setting.json`](https://github.com/apache/zeppelin/blob/master/spark/src/main/resources/interpreter-setting.json) file? :)
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: Commented the properties in interpreter file and added...
Posted by jongyoul <gi...@git.apache.org>.
Github user jongyoul commented on the issue:
https://github.com/apache/zeppelin/pull/1548
Thanks for the contribution. I left a comment and please check 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1548
@meenakshisekar Could you take care of [this](https://github.com/apache/zeppelin/pull/1548#discussion_r84897785)?
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar closed the pull request at:
https://github.com/apache/zeppelin/pull/1548
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1548
Seems the reason of CI failure is irrelevant with this change. Several tests are failed because of Livy.
```
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Zeppelin ........................................... SUCCESS [ 6.867 s]
[INFO] Zeppelin: Interpreter .............................. SUCCESS [02:03 min]
[INFO] Zeppelin: Zengine .................................. SUCCESS [01:05 min]
[INFO] Zeppelin: Display system apis ...................... SUCCESS [ 3.410 s]
[INFO] Zeppelin: Spark dependencies ....................... SUCCESS [ 55.243 s]
[INFO] Zeppelin: Spark .................................... SUCCESS [01:53 min]
[INFO] Zeppelin: Markdown interpreter ..................... SUCCESS [ 1.399 s]
[INFO] Zeppelin: Angular interpreter ...................... SUCCESS [ 0.311 s]
[INFO] Zeppelin: Shell interpreter ........................ SUCCESS [ 0.632 s]
[INFO] Zeppelin: Livy interpreter ......................... FAILURE [02:13 min]
[INFO] Zeppelin: HBase interpreter ........................ SKIPPED
[INFO] Zeppelin: Apache Pig Interpreter ................... SKIPPED
[INFO] Zeppelin: PostgreSQL interpreter ................... SKIPPED
[INFO] Zeppelin: JDBC interpreter ......................... SKIPPED
[INFO] Zeppelin: File System Interpreters ................. SKIPPED
[INFO] Zeppelin: Flink .................................... SKIPPED
[INFO] Zeppelin: Apache Ignite interpreter ................ SKIPPED
[INFO] Zeppelin: Kylin interpreter ........................ SKIPPED
[INFO] Zeppelin: Python interpreter ....................... SKIPPED
[INFO] Zeppelin: Lens interpreter ......................... SKIPPED
[INFO] Zeppelin: Apache Cassandra interpreter ............. SKIPPED
[INFO] Zeppelin: Elasticsearch interpreter ................ SKIPPED
[INFO] Zeppelin: BigQuery interpreter ..................... SKIPPED
[INFO] Zeppelin: Alluxio interpreter ...................... SKIPPED
[INFO] Zeppelin: web Application .......................... SKIPPED
[INFO] Zeppelin: Server ................................... SKIPPED
[INFO] Zeppelin: Packaging distribution ................... SKIPPED
[INFO] Zeppelin: Scalding interpreter ..................... SKIPPED
[INFO] Zeppelin: Examples ................................. SKIPPED
[INFO] Zeppelin: Example application - Clock .............. SKIPPED
[INFO] Zeppelin: Example application - Horizontal Bar chart SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 08:24 min
[INFO] Finished at: 2016-11-01T18:59:03+00:00
[INFO] Final Memory: 62M/407M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-remote-resources-plugin:1.5:process (default) on project zeppelin-livy: Error resolving project artifact: Could not transfer artifact org.mortbay.jetty:jetty:pom:6.1.26.cloudera.4 from/to sonatype-apache (https://repository.apache.org/releases/): Connect to repository.apache.org:443 [repository.apache.org/207.244.88.143] failed: Connection timed out for project org.mortbay.jetty:jetty:jar:6.1.26.cloudera.4 ->
```
@meenakshisekar The latest change is looks good to me. Just in case, could you retrigger CI again? At least Livy won't be the failure reason.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
GitHub user meenakshisekar reopened a pull request:
https://github.com/apache/zeppelin/pull/1548
[ZEPPELIN-917] Apply new mechanism to LensInterpreter
### What is this PR for?
This handles replacing the registration of interpreter with static block by the interpreter-setting.json file
### What type of PR is it?
[| Improvement |
### Todos
Sub-Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-917
### How should this be tested?
here shouldn't be any warning like below on starting the server
INFO [2016-09-29 00:25:46,247] ({main} LensInterpreter.java[<clinit>]:155) - Bootstrapping Cassandra Interpreter WARN [2016-09-29 00:25:46,250] ({main} Interpreter.java[register]:347) - Static initialization is deprecated for interpreter lens, You should change it to use interpreter-setting.json in your jar or interpreter/{interpreter}/interpreter-setting.json INFO [2016-09-29 00:25:46,250] ({main} InterpreterFactory.java[init]:204) - Inclass=org.apache.zeppelin.lens.LensInterpreter
And ensure that the lens related paragraphs run without any error
### 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/meenakshisekar/zeppelin ZEPPELIN-917
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1548.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 #1548
----
commit db2827f867199cdd5e3fe2bc05f614dd88e013f5
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T09:59:38Z
Commented the properties in interpreter file and added setting file.
commit f94c8a3d44d441174fff2e89708e9ec62c1f5d22
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T11:41:58Z
Commented the properties in interpreter file and added setting file.
commit ad3b83ca856c54727f1ffef324f187f69a276c83
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-24T11:57:46Z
Added indentation
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
GitHub user meenakshisekar reopened a pull request:
https://github.com/apache/zeppelin/pull/1548
[ZEPPELIN-917] Apply new mechanism to LensInterpreter
### What is this PR for?
This handles replacing the registration of interpreter with static block by the interpreter-setting.json file
### What type of PR is it?
[| Improvement |
### Todos
Sub-Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-917
### How should this be tested?
here shouldn't be any warning like below on starting the server
INFO [2016-09-29 00:25:46,247]({main} LensInterpreter.java[<clinit>]:155) - Bootstrapping Cassandra Interpreter WARN [2016-09-29 00:25:46,250]({main} Interpreter.java[register]:347) - Static initialization is deprecated for interpreter lens, You should change it to use interpreter-setting.json in your jar or interpreter/{interpreter}/interpreter-setting.json INFO [2016-09-29 00:25:46,250]({main} InterpreterFactory.java[init]:204) - Inclass=org.apache.zeppelin.lens.LensInterpreter
And ensure that the lens related paragraphs run without any error
### 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/meenakshisekar/zeppelin ZEPPELIN-917
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1548.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 #1548
----
commit db2827f867199cdd5e3fe2bc05f614dd88e013f5
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T09:59:38Z
Commented the properties in interpreter file and added setting file.
commit f94c8a3d44d441174fff2e89708e9ec62c1f5d22
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T11:41:58Z
Commented the properties in interpreter file and added setting file.
commit ad3b83ca856c54727f1ffef324f187f69a276c83
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-24T11:57:46Z
Added indentation
commit cc8bdc274f581e7725307e2a4706b4d8ce4e243c
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T07:12:54Z
Added the indentation similar to other interpreter setting file.
commit 2cba9e74df90073bd03c2867a70c345e5346dc7e
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T07:12:54Z
Added the indentation similar to other interpreter setting file.
commit 269e8cb3c547d5f1ea932c571206c13931a462ca
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T08:02:10Z
Removed the tabs and added space.
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar commented on the issue:
https://github.com/apache/zeppelin/pull/1548
@AhyoungRyu
I changed the indentation comment and checked the changes in the file again.
Do you mean something different.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar commented on the issue:
https://github.com/apache/zeppelin/pull/1548
Reopening for travis build.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar closed the pull request at:
https://github.com/apache/zeppelin/pull/1548
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar commented on the issue:
https://github.com/apache/zeppelin/pull/1548
reopening for Travis build.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by AhyoungRyu <gi...@git.apache.org>.
Github user AhyoungRyu commented on the issue:
https://github.com/apache/zeppelin/pull/1548
@meenakshisekar Nice!
Will merge this if there are no more comments.
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin issue #1548: [ZEPPELIN-917] Apply new mechanism to LensInterpreter
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar commented on the issue:
https://github.com/apache/zeppelin/pull/1548
CI Build is green.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar closed the pull request at:
https://github.com/apache/zeppelin/pull/1548
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
GitHub user meenakshisekar reopened a pull request:
https://github.com/apache/zeppelin/pull/1548
[ZEPPELIN-917] Apply new mechanism to LensInterpreter
### What is this PR for?
This handles replacing the registration of interpreter with static block by the interpreter-setting.json file
### What type of PR is it?
[| Improvement |
### Todos
Sub-Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-917
### How should this be tested?
here shouldn't be any warning like below on starting the server
INFO [2016-09-29 00:25:46,247]({main} LensInterpreter.java[<clinit>]:155) - Bootstrapping Cassandra Interpreter WARN [2016-09-29 00:25:46,250]({main} Interpreter.java[register]:347) - Static initialization is deprecated for interpreter lens, You should change it to use interpreter-setting.json in your jar or interpreter/{interpreter}/interpreter-setting.json INFO [2016-09-29 00:25:46,250]({main} InterpreterFactory.java[init]:204) - Inclass=org.apache.zeppelin.lens.LensInterpreter
And ensure that the lens related paragraphs run without any error
### 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/meenakshisekar/zeppelin ZEPPELIN-917
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1548.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 #1548
----
commit db2827f867199cdd5e3fe2bc05f614dd88e013f5
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T09:59:38Z
Commented the properties in interpreter file and added setting file.
commit f94c8a3d44d441174fff2e89708e9ec62c1f5d22
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T11:41:58Z
Commented the properties in interpreter file and added setting file.
commit ad3b83ca856c54727f1ffef324f187f69a276c83
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-24T11:57:46Z
Added indentation
commit 2cba9e74df90073bd03c2867a70c345e5346dc7e
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T07:12:54Z
Added the indentation similar to other interpreter setting file.
commit cc8bdc274f581e7725307e2a4706b4d8ce4e243c
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T07:12:54Z
Added the indentation similar to other interpreter setting file.
commit 269e8cb3c547d5f1ea932c571206c13931a462ca
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-27T08:02:10Z
Removed the tabs and added space.
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
Github user meenakshisekar closed the pull request at:
https://github.com/apache/zeppelin/pull/1548
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] zeppelin pull request #1548: [ZEPPELIN-917] Apply new mechanism to LensInter...
Posted by meenakshisekar <gi...@git.apache.org>.
GitHub user meenakshisekar reopened a pull request:
https://github.com/apache/zeppelin/pull/1548
[ZEPPELIN-917] Apply new mechanism to LensInterpreter
### What is this PR for?
This handles replacing the registration of interpreter with static block by the interpreter-setting.json file
### What type of PR is it?
[| Improvement |
### Todos
Sub-Task
### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-917
### How should this be tested?
here shouldn't be any warning like below on starting the server
INFO [2016-09-29 00:25:46,247] ({main} LensInterpreter.java[<clinit>]:155) - Bootstrapping Cassandra Interpreter WARN [2016-09-29 00:25:46,250] ({main} Interpreter.java[register]:347) - Static initialization is deprecated for interpreter lens, You should change it to use interpreter-setting.json in your jar or interpreter/{interpreter}/interpreter-setting.json INFO [2016-09-29 00:25:46,250] ({main} InterpreterFactory.java[init]:204) - Inclass=org.apache.zeppelin.lens.LensInterpreter
And ensure that the lens related paragraphs run without any error
### 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/meenakshisekar/zeppelin ZEPPELIN-917
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/zeppelin/pull/1548.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 #1548
----
commit db2827f867199cdd5e3fe2bc05f614dd88e013f5
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T09:59:38Z
Commented the properties in interpreter file and added setting file.
commit f94c8a3d44d441174fff2e89708e9ec62c1f5d22
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-21T11:41:58Z
Commented the properties in interpreter file and added setting file.
commit ad3b83ca856c54727f1ffef324f187f69a276c83
Author: meenakshisekar <me...@imaginea.com>
Date: 2016-10-24T11:57:46Z
Added indentation
----
---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---