date:20170125

[GitHub] zeppelin pull request #1947: [HOTFIX][ZEPPELIN-2013] Exceeded size of logs m...

2017-01-25 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #1947: [HOTFIX][ZEPPELIN-2013] Exceeded size of logs makes CI...

2017-01-25 Thread jongyoul

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1947
  
Will merge it onto master and branch-0.7


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1945: [ZEPPELIN-2003] Remove PostgresqlInterpreter

2017-01-25 Thread jongyoul

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1945
  
@felixcheung Sounds like a good idea!


---
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 #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread del680202

Github user del680202 commented on the issue:

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


@zjffdu 
I  verify this issue again for double check.

Livy version: branch-0.3

There are two test cases:
1. Select temp table from spark
2. Select real table from hive


In the previous, My PR occur problem because livy interpreter use '\n' for 
comment part but it doesn't filter out '\n'.
Therefore, I fix it at first.
I suppose it is a policy problem, does '\n' in interpreter result need to 
be filtered out?

Another problem
In master branch, when my target is hive table, it will show ``1`` in 
result.

-master branch
![2017-01-26 13 58 
48](https://cloud.githubusercontent.com/assets/3747345/22320624/336225a2-e3d1-11e6-8d0a-f0cdcf51023b.png)


-This PR(after fixing)
![2017-01-26 14 01 
41](https://cloud.githubusercontent.com/assets/3747345/22320627/39904166-e3d1-11e6-9a0e-441bef44f7bb.png)




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1742: [ZEPPELIN-1781] Failed to run the script 'test_zeppeli...

2017-01-25 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1742
  
sorry I didnt see this. LGTM, CI passed, 
merging if no more comment


---
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 #1945: [ZEPPELIN-2003] Remove PostgresqlInterpreter

2017-01-25 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1945
  
shall we leave the postgreSQL interpreter md file to point to JDBC one? I 
think our users might be looking for it


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1943: [HOTFIX][ZEPPELIN-2011] Add lib/interpreter to classpa...

2017-01-25 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1943
  
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 #1947: [HOTFIX][ZEPPELIN-2013] Exceeded size of logs makes CI...

2017-01-25 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1947
  
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 #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread del680202

Github user del680202 commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
@zjffdu 
I  verify this issue again for double check. 
This PR working well, Maybe I miss something.
But master branch still occur problem.

There are two test cases:
* Select table from spark temp table
* Select real table from hive

If my target is hive table, it will show ``1`` in master branch.

-master branch
![2017-01-26 13 58 
48](https://cloud.githubusercontent.com/assets/3747345/22320624/336225a2-e3d1-11e6-8d0a-f0cdcf51023b.png)


-This PR
![2017-01-26 14 01 
41](https://cloud.githubusercontent.com/assets/3747345/22320627/39904166-e3d1-11e6-9a0e-441bef44f7bb.png)





---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread felixcheung

Github user felixcheung commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
I think we need this for 0.7 RC?


---
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 #1943: [HOTFIX][ZEPPELIN-2011] Add lib/interpreter to ...

2017-01-25 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #1941: [ZEPPELIN-2009] Cron job isn't executed after c...

2017-01-25 Thread asfgit

Github user asfgit closed the pull request at:

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


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

[CANCEL][VOTE] Release Apache Zeppelin 0.7.0 (RC3)

2017-01-25 Thread Mina Lee

There are several critical issues found during the rc3 verification.
I am canceling vote to include below bug fixes:
   - Cron scheduling doesn't work after certain number of running [1]
   - Table is not rendered in certain condition [2]
   - Livy not working in anonymous mode [3]
   - Interpreter install script throws "NoClassDefFoundError" [4]

[1] https://github.com/apache/zeppelin/pull/1941
[2] https://github.com/apache/zeppelin/pull/1942
[3] https://github.com/apache/zeppelin/pull/1938
[4] https://github.com/apache/zeppelin/pull/1943

On Wed, Jan 25, 2017 at 3:44 PM Jeff Zhang  wrote:

> And seems ZEPPELIN-1982 also need to be included in 0.7
>
>
> Jeff Zhang 于2017年1月25日周三 下午1:38写道：
>
> Please help also merge ZEPPELIN-2006 (
> https://github.com/apache/zeppelin/pull/1938) If there's another RC
>
>
> moon soo Lee 于2017年1月25日周三 上午11:22写道：
>
> Currently, cron scheduling feature is broken [1] and patch is available at
> [2].
> Shell we include this patch in 0.7.0 release?
>
> [1] http://issues.apache.org/jira/browse/ZEPPELIN-2009
> [2] https://github.com/apache/zeppelin/pull/1941
>
> On Tue, Jan 24, 2017 at 6:45 PM Prabhjyot Singh 
> wrote:
>
> +1
>
> On Jan 24, 2017 6:25 PM, "Ahyoung Ryu"  wrote:
>
> +1
>
> On Tue, Jan 24, 2017 at 8:19 PM, Hyung Sung Shim 
> wrote:
>
> +1
> 2017년 1월 24일 (화) 오전 1:36, Windy Qin 님이 작성:
>
>
> +1
> On 2017-01-24 17:13 (+0800), Sora Lee  wrote:
> > +1
> >
> > On Tue, Jan 24, 2017 at 3:09 PM Jongyoul Lee  wrote:
> >
> > > +1 (binding)
> > >
> > > On Tue, Jan 24, 2017 at 2:43 PM, Mina Lee  wrote:
> > >
> > > I propose the following RC to be released for the Apache Zeppelin 0.7.0
> > > release.
> > >
> > > The commit id is 48ad70e8c62975bdb00779bed5919eaca98c5b5d which is
> > > corresponds to the tag v0.7.0-rc3:
> > > *
> https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=48ad70e8c62975bdb00779bed5919eaca98c5b5d
> > > <
> https://git-wip-us.apache.org/repos/asf?p=zeppelin.git;a=commit;h=48ad70e8c62975bdb00779bed5919eaca98c5b5d
> >*
> > >
> > > The release archives (tgz), signature, and checksums are here
> > > https://dist.apache.org/repos/dist/dev/zeppelin/zeppelin-0.7.0-rc3/
> > >
> > > The release candidate consists of the following source distribution
> > > archive
> > > zeppelin-0.7.0.tgz
> > >
> > > In addition, the following supplementary binary distributions are
> provided
> > > for user convenience at the same location
> > > zeppelin-0.7.0-bin-all.tgz
> > > zeppelin-0.7.0-bin-netinst.tgz
> > >
> > > The maven artifacts are here
> > >
> https://repository.apache.org/content/repositories/orgapachezeppelin-1024
> > >
> > > You can find the KEYS file here:
> > > https://dist.apache.org/repos/dist/release/zeppelin/KEYS
> > >
> > > Release notes available at
> > >
> > >
> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12336544=12316221
> > >
> > > Vote will be open for next 72 hours (close at 22:00 26/Jan PST).
> > >
> > > [ ] +1 approve
> > > [ ] 0 no opinion
> > > [ ] -1 disapprove (and reason why)
> > >
> > >
> > >
> > >
> > > --
> > > 이종열, Jongyoul Lee, 李宗烈
> > > http://madeng.net
> > >
> >
>
>
>

[GitHub] zeppelin issue #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread del680202

Github user del680202 commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
I test Jdbc interpreter and livy interpreter by my test database.

jdbc interpreter working well.
![2017-01-26 11 32 
07](https://cloud.githubusercontent.com/assets/3747345/22318534/cb8a7c70-e3be-11e6-88f6-d44dd074619d.png)

But there a problem in livy interpreter 
![2017-01-26 11 33 
56](https://cloud.githubusercontent.com/assets/3747345/22318560/f552e1f0-e3be-11e6-8fd8-190612a80af9.png)

And then, I test livy interpreter  by master branch again, the same problem 
occurred
I suppose it is a livy interpreter  issue.

![2017-01-26 12 02 
31](https://cloud.githubusercontent.com/assets/3747345/22318616/67c8bbce-e3bf-11e6-83bb-c57c49456a89.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 #1938: ZEPPELIN-2006. Livy interpreter doesn't work in...

2017-01-25 Thread asfgit

Github user asfgit closed the pull request at:

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


---
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 #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
Jdbc interpreter may be effected. Livy interpreter would return a list of 
InterpreterResult so it is fine for livy. 


---
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 #1944: [HOTFIX][MINOR] Change the scope of httpclient to runt...

2017-01-25 Thread jongyoul

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1944
  
My command only includes 
`/Users/jl/.m2/repository/commons-httpclient/commons-httpclient/3.1/commons-httpclient-3.1.jar`
 which is different from `org.apache.httpclient`. Can you show me your command 
for debugger?


---
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-2014) Jetty Directory Listing on app, assets, components, and scripts

2017-01-25 Thread Ian Tyndall (JIRA)

Ian Tyndall created ZEPPELIN-2014:
-

 Summary: Jetty Directory Listing on app, assets, components, and 
scripts
 Key: ZEPPELIN-2014
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2014
 Project: Zeppelin
  Issue Type: Bug
  Components: Core
Affects Versions: 0.6.2
 Environment: RHEL
Reporter: Ian Tyndall


Security Issue, would not pass my institution's security scanners.
The Web directory list is made publicly accessible folders by default.
As a bandaid, I've added code in the daemon shell script to put index html 
files with a meta refresh in the affected directories.

It would be nice if this could be configured on the fly with other jetty config 
with this:
https://www.eclipse.org/jetty/documentation/9.3.x/override-web-xml.html

But, a nice hard coded fix would be great in the meantime!



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

[GitHub] zeppelin pull request #1947: [HOTFIX][ZEPPELIN-2013] Exceeded size of logs m...

2017-01-25 Thread jongyoul

GitHub user jongyoul opened a pull request:

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

[HOTFIX][ZEPPELIN-2013] Exceeded size of logs makes CI failed

### What is this PR for?
avoiding failure of CI due to exceeded limit of log size. 
https://s3.amazonaws.com/archive.travis-ci.org/jobs/195087823/log.txt

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

### Todos
* [x] - Changed level of logs 

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

### How should this be tested?

### 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-2013

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

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


commit 75d1edcef5e5db9b1404015261a9d65b894bbab3
Author: Jongyoul Lee 
Date:   2017-01-25T16:23:54Z

Chagned logger level to debug from info




---
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-2013) Exceeded logs fails CI

2017-01-25 Thread Jongyoul Lee (JIRA)

Jongyoul Lee created ZEPPELIN-2013:
--

 Summary: Exceeded logs fails CI
 Key: ZEPPELIN-2013
 URL: https://issues.apache.org/jira/browse/ZEPPELIN-2013
 Project: Zeppelin
  Issue Type: Sub-task
Reporter: Jongyoul Lee
Assignee: Jongyoul Lee


09:40:51,817  INFO org.apache.zeppelin.dep.DependencyResolver:102 - copy 
/home/travis/build/apache/zeppelin/zeppelin-server/../local-repo/org/apache/drill/exec/vector/1.6.0/vector-1.6.0.jar
 to ../local-repo/2C9CY118N
09:40:51,818  INFO org.apache.zeppelin.dep.DependencyResolver:102 - copy 
/home/travis/build/apache/zeppelin/zeppelin-server/../local-repo/com/beust/jcommander/1.30/jcommander-1.30.jar
 to ../local-repo/2C9CY118N
09:40:51,819  INFO org.apache.zeppelin.dep.DependencyResolver:102 - copy 
/home/travis/build/apache/zeppelin/zeppelin-server/../local-repo/org/apache/curator/curator-x-discovery/2.7.1/curator-x-discovery-2.7.1.jar
 to ../local-repo/2C9CY118N
09:40:51,819  INFO org.apache.zeppelin.dep.DependencyResolver:102 - copy 
/home/travis/build/apache/zeppelin/zeppelin-server/../local-repo/org/apache/curator/curator-recipes/2.7.1/curator-recipes-2.7.1.jar
 to ../local-repo/2C9CY118N
09:40:51,820  INFO org.apache.zeppelin.dep.DependencyResolver:102 - copy 
/home/travis/build/apache/zeppelin/zeppelin-server/../local-repo/org/apache/curator/curator-framework/2.7.1/curator-framework-2.7.1.jar
 to ../local-repo/2C9CY118N


The log length has exceeded the limit of 4 MB (this usually means that the test 
suite is raising the same exception over and over).

The job has been terminated



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

[GitHub] zeppelin issue #1893: [ZEPPELIN-451] Save codes and messages as multi-line

2017-01-25 Thread soralee

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1893
  
Thank @sixmen for improving this. It's very good feature!

I think sudden changes, part of importing notebook, will make users 
uncomfortable. 
In my opinion, both old version notebook and now version notebook should be 
nice if it is compatible.


---
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 #1944: [HOTFIX][MINOR] Change the scope of httpclient to runt...

2017-01-25 Thread jongyoul

Github user jongyoul commented on the issue:

https://github.com/apache/zeppelin/pull/1944
  
Yes, Sure. But when IntelliJ runs ZeppelinServer in debug mode, actual 
command is what I mentioned before which doesn't include any httpclient 
libraries but include zeppelin-{zengine,interpreter}/target/classes. I guess 
IntelliJ select and compile its dependencies inside of project.

I just wonder why you can run ZeppelinServer in IntelliJ without any 
specific settings. Here's my configuration. Can you check it?
https://cloud.githubusercontent.com/assets/3612566/22297717/869e3050-e361-11e6-974b-a917449a3189.png;>



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread soralee

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
 @del680202 I tested this again and I can't the above error in chrome.
It's working well now.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1944: [HOTFIX][MINOR] Change the scope of httpclient to runt...

2017-01-25 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1944
  
I don't have special configuration for Intellij but does not have any 
problem after Helium.

If you run following command, you'll see all dependencies of 
`zeppelin-server` module
```
mvn -DskipTests -Pspark-2.1 -Pscala-2.11 -Phadoop-2.6 -Pyarn -Ppyspark -pl 
'zeppelin-interpreter,zeppelin-zengine,zeppelin-server' package dependency:tree
```

And there're `httpclient` dependency in `compile` scope. For example

```
[INFO] org.apache.zeppelin:zeppelin-interpreter:jar:0.8.0-SNAPSHOT
[INFO] +- org.apache.thrift:libthrift:jar:0.9.2:compile
[INFO] |  +- org.apache.httpcomponents:httpclient:jar:4.3.6:compile
[INFO] |  |  +- commons-logging:commons-logging:jar:1.1.1:compile
[INFO] |  |  \- commons-codec:commons-codec:jar:1.5:compile
[INFO] |  \- org.apache.httpcomponents:httpcore:jar:4.3.3:compile
```

and

```
[INFO] +- org.apache.maven.wagon:wagon-http:jar:1.0:compile
[INFO] |  \- org.apache.maven.wagon:wagon-http-shared:jar:1.0:compile
[INFO] | +- commons-httpclient:commons-httpclient:jar:3.1:compile
[INFO] | +- commons-io:commons-io:jar:2.4:compile
[INFO] | \- org.jsoup:jsoup:jar:1.6.1:compile
```

```
[INFO] +- org.apache.httpcomponents:httpclient:jar:4.3.6:compile
[INFO] |  +- org.apache.httpcomponents:httpcore:jar:4.3.3:compile
[INFO] |  \- commons-codec:commons-codec:jar:1.5:compile
```

Yes, `httpclient` is not compile scope dependency in `zeppelin-server` 
module. But it's compile scope dependency in `zeppelin-interpreter` and 
`zeppelin-zengine` module. And `zeppelin-server` have `zeppelin-interpreter` 
and `zeppelin-zengine` as a dependency. So `zeppelin-server` should have 
`httpclient` in it's compile scope dependency when try to run in intellij.

What do you think?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread Leemoonsoo

Github user Leemoonsoo commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
Thanks @del680202  for the fix!

Although it fixes problem from of output from SparkInterpreter, outputs 
from other interpreters may have the same problem unless other interpreter 
changes code.

So, I think we need follow up PR that fixes InterpreterOutput.java 's table 
data comment handling.


---
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 #1943: [HOTFIX][ZEPPELIN-2011] Add lib/interpreter to classpa...

2017-01-25 Thread soralee

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1943
  
Tested and it was working well, 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 #1941: [ZEPPELIN-2009] Cron job isn't executed after couple o...

2017-01-25 Thread khalidhuseynov

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1941
  
@minahlee  yeah definitely, i was also thinking to have test case as a 
follow up


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

RE: Build on Windows getting stuck

2017-01-25 Thread Igor Drozdov

I have the same issue with npm-run-all clean on windows 10

Igor Drozdov

From: Andrii Pelekh [mailto:pelek...@gmail.com]
Sent: Wednesday, January 25, 2017 7:56 AM
To: dev@zeppelin.apache.org
Subject: Build on Windows getting stuck

Hi all,

I'm trying to reproduce a defect from Jira ticket on Windows7 x64 but my build 
getting stuck on the "npm-run-all clean" step. Please see log file attached. I 
tried to build on another Windows PC and it got stuck too on the same step. I 
have dual boot and everything works fine for me on Ubuntu.

Another thing to note is two node processes which I observe during the hang up 
phase (see screenshot attached). One of the processes (yarn.js run build) does 
not terminate even after I cancel the build.

Have anyone had anything similar?

Thanks,
Andrii

[GitHub] zeppelin issue #1941: [ZEPPELIN-2009] Cron job isn't executed after couple o...

2017-01-25 Thread minahlee

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1941
  
@khalidhuseynov I am going to merge this one in few hours, but do you mind 
to create test cast for this in another PR?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1941: [ZEPPELIN-2009] Cron job isn't executed after couple o...

2017-01-25 Thread minahlee

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1941
  
Tested and CI is green now. Merge to branch-0.7 and master if there is no 
more 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 #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread del680202

Github user del680202 commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
@soralee @zjffdu
I also figure out that i missed the last new line character, i use the a 
simple way to handle it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread zjffdu

Github user zjffdu commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
@soralee I can see the same error in chrome. 


---
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 #1942: [ZEPPELIN-1982] When using the 'Select * ...' statemen...

2017-01-25 Thread soralee

Github user soralee commented on the issue:

https://github.com/apache/zeppelin/pull/1942
  
Thank @del680202 for fixing this bug!

I just tested it out and this problem works well.
But, some existing sample paragraphs have the following problems. 

![image](https://cloud.githubusercontent.com/assets/8110458/22285104/9e2982bc-e32d-11e6-9f3e-79b834e1f9fb.png)

and, when I run one paragraph among those paragraph

![z_1942](https://cloud.githubusercontent.com/assets/8110458/22283683/00a64b52-e327-11e6-8b26-4bb9f06e4f58.gif)


@del680202 Could you check and run this paragraph?



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] zeppelin issue #1941: [ZEPPELIN-2009] Cron job isn't executed after couple o...

2017-01-25 Thread khalidhuseynov

Github user khalidhuseynov commented on the issue:

https://github.com/apache/zeppelin/pull/1941
  
@minahlee sorry for that, just addressed


---
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 #1946: [ZEPPELIN-921] Apply new mechanism to KnitR and RRepl

2017-01-25 Thread DrIgor

Github user DrIgor commented on the issue:

https://github.com/apache/zeppelin/pull/1946
  
Needs this changes: #1937


---
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 #1946: [ZEPPELIN-921] Apply new mechanism to KnitR and...

2017-01-25 Thread DrIgor

GitHub user DrIgor opened a pull request:

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

[ZEPPELIN-921] Apply new mechanism to KnitR and RRepl

### What is this PR for?
Applies new registration mechanism for R interpreter

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

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

### How should this be tested?
Build zeppelin with R profile enabled and run several R snippets

### 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/DrIgor/zeppelin ZEPPELIN-921

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

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


commit b1dd6a33040b17be7cd52a8c482062453a00ed44
Author: Igor Drozdov 
Date:   2017-01-23T11:20:38Z

Search for interpreter-settings in several jars in folder

commit b264c9fff7ea2f721b64eab31fae9abf7c9ba107
Author: Igor Drozdov 
Date:   2017-01-25T08:36:53Z

New registration mechanism for R interpreter




---
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 #1941: [ZEPPELIN-2009] Cron job isn't executed after couple o...

2017-01-25 Thread minahlee

Github user minahlee commented on the issue:

https://github.com/apache/zeppelin/pull/1941
  
Another test is failing in multiple profiles, do you mind to fix them too?
```
Results :

Failed tests: 
  NotebookRestApiTest.testRunParagraphJob:108 expected: but 
was:

Tests run: 82, Failures: 1, Errors: 0, Skipped: 1
```


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

37 matches

Mail list logo