[GitHub] zeppelin issue #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2199 Merge into master and branch-0.7 if there are no more comments on 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2199 CI is green now. 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user markito commented on the issue: https://github.com/apache/zeppelin/pull/2199 No problem, just reopened --- 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2199 @markito I mean, this PR not the JIRA ticket :) --- 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user markito commented on the issue: https://github.com/apache/zeppelin/pull/2199 @1ambda can you please confirm if the reopened JIRA triggered Jenkins properly ? Thanks! --- 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2199 Great! Then, Please close issue and re-open :) then jenkins will be triggered 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user markito commented on the issue: https://github.com/apache/zeppelin/pull/2199 @1ambda indeed! I've restarted the CI job and now it's all green! https://travis-ci.org/markito/zeppelin/builds/216598188 --- 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user 1ambda commented on the issue: https://github.com/apache/zeppelin/pull/2199 I am not sure but the failed test is due to one of flaky tests. Please restart it. ``` Results : Tests in error: InterpreterIT.testShowDescriptionOnInterpreterCreate:69 û ElementNotVisible El... ``` --- 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user markito commented on the issue: https://github.com/apache/zeppelin/pull/2199 I'm sorry, skipped that part from the contribution guide. Will do and amend. Thanks! --- 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user AhyoungRyu commented on the issue: https://github.com/apache/zeppelin/pull/2199 Hi @markito Thanks for your contribution and taking care of Geode interpreter. - Zeppelin build system relies on travis. So please setup your travis account and travis project for Zeppelin. - Please take a look https://zeppelin.apache.org/contribution/contributions.html#continuous-integration - The repository name must be Zeppelin (not incubator-zeppelin, ...) - It's not mandatory to pass all tests to be reviewed. But some reviewers might select passed PRs first Trigger travis CI by pushing new commits or amend your last commit (git commit --amend and git push your-remote HEAD --force) after setup your travis. --- 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 #2199: [ZEPPELIN-2326] - Updating Geode dependencies and impo...
Github user felixcheung commented on the issue: https://github.com/apache/zeppelin/pull/2199 looks good, could you please setup CI as referenced here https://zeppelin.apache.org/contribution/contributions.html#continuous-integration --- 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. ---