[GitHub] zeppelin issue #2282: [ZEPPELIN-2447] Fix python interpreter as to use max r...

2017-06-09 Thread paularmand
Github user paularmand commented on the issue: https://github.com/apache/zeppelin/pull/2282 Pong @jongyoul . I wrote the unit test, but it is not passing on Jenkins. I have trouble setting up my local IDE to run the python tests in IntelliJ (all other tests, except the python ones

[GitHub] zeppelin issue #2282: [ZEPPELIN-2447] Fix python interpreter as to use max r...

2017-05-03 Thread paularmand
Github user paularmand commented on the issue: https://github.com/apache/zeppelin/pull/2282 @zjffdu I'll have a look. Can expect something later on this week. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] zeppelin issue #2282: [ZEPPELIN-2447] Fix python interpreter as to use max r...

2017-05-02 Thread paularmand
Github user paularmand commented on the issue: https://github.com/apache/zeppelin/pull/2282 ping @zjffdu Do I have to look into it the Selenium test failure? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] zeppelin issue #2282: [ZEPPELIN-2447] Fix python interpreter as to use max r...

2017-04-28 Thread paularmand
Github user paularmand commented on the issue: https://github.com/apache/zeppelin/pull/2282 Is the Jenkins failure an unrelated selenium test failure, or do I have to look into it? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] zeppelin issue #2282: Fix python interpreter as to use max result setting

2017-04-28 Thread paularmand
Github user paularmand commented on the issue: https://github.com/apache/zeppelin/pull/2282 @zjffdu Thanks for review. Added 1000 hard coded as default for now, although I believe a better solution would be to add fallbacks to a property class upfront (see https://docs.oracle.com

[GitHub] zeppelin issue #2282: Fix python interpreter as to use max result setting

2017-04-25 Thread paularmand
Github user paularmand commented on the issue: https://github.com/apache/zeppelin/pull/2282 Ready for review --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes

[GitHub] zeppelin pull request #2282: fix python interpreter as to use max result set...

2017-04-25 Thread paularmand
GitHub user paularmand opened a pull request: https://github.com/apache/zeppelin/pull/2282 fix python interpreter as to use max result setting ### What is this PR for? zeppelin_python.py used a hard-coded 1000 rows limit when returning results. This PR makes sure that the