Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2645
@yiming187 I have made the changes and updated pull request. You can
review now.
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2645
@zjffdu Thanks for your comments and review. There seems something wrong
with validating pull request.
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2645
@zjffdu the CI checks seems to get stuck, even after rebase.
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2645
@zjffdu please review. Thanks.
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2645
@zjffdu sure, kylin interpreter has unit test class, and I will add the
test when result set is empty.
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2645
@zjffdu @felixcheung can you please review? I have rebased and created this
new pull request. [was https://github.com/apache/zeppelin/pull/2634]. Thanks
for your time!
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2634
@zjffdu sure I will close this PR and have created another PR
https://github.com/apache/zeppelin/pull/2645. Please review.
---
Github user jinxliu closed the pull request at:
https://github.com/apache/zeppelin/pull/2634
---
GitHub user jinxliu opened a pull request:
https://github.com/apache/zeppelin/pull/2645
ZEPPELIN-3014: NPE bug fix and Error message enhancement with Kylin Iâ¦
â¦nterpreter
### What is this PR for?
A few sentences describing the overall goals of the pull request&#
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2634
@felixcheung it still does not work.
---
GitHub user jinxliu reopened a pull request:
https://github.com/apache/zeppelin/pull/2634
[ZEPPELIN-3014] NPE bug fix and Error message enhancement with Kylin
Interpreter
### What is this PR for?
A few sentences describing the overall goals of the pull request's co
Github user jinxliu closed the pull request at:
https://github.com/apache/zeppelin/pull/2634
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2634
I have tried several times since yesterday, hope this time it works.
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2634
@felixcheung is there something wrong with travis-ci?
---
Github user jinxliu commented on the issue:
https://github.com/apache/zeppelin/pull/2634
This PR has two commits,
https://github.com/apache/zeppelin/pull/2634/commits/ec68d12437611dd6fb817fa058020bb926b7e163
is to enhance the error message parse and display, the second commit is
GitHub user jinxliu opened a pull request:
https://github.com/apache/zeppelin/pull/2634
Kylin intp
### What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide -
https://zeppelin.apach
16 matches
Mail list logo
