[GitHub] zeppelin issue #918: ZEPPELIN-885 Add option to send email in scheduler exec...

Github user vgmartinez commented on the issue: https://github.com/apache/zeppelin/pull/918 Hi @Leemoonsoo, I have refactored the code to be able to add more channels of notifications, e.g slack, SNS (in the next PR)... Please take a look... thks --- If your project

[GitHub] zeppelin issue #998: Authenticated user aware notebook storage layer

Github user vgmartinez commented on the issue: https://github.com/apache/zeppelin/pull/998 LGTM...I have tested and works well for me. --- 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

[GitHub] zeppelin issue #918: ZEPPELIN-885 Add option to send email in scheduler exec...

Github user vgmartinez commented on the issue: https://github.com/apache/zeppelin/pull/918 @Leemoonsoo CI check... --- 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

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

Github user vgmartinez commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 @jongyoul ok ok, all external dependencies loaded them that way. I have been a time out...;) --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-zeppelin issue #943: ZEPPELIN-925 Merge HiveInterpreter into JDBCI...

Github user vgmartinez commented on the issue: https://github.com/apache/incubator-zeppelin/pull/943 Hi @jongyoul, great job with all integrations in the jdbc interpreter. What do you think, about putting hive dependence into the pom and so it is not necessary add the jar

[GitHub] incubator-zeppelin issue #918: ZEPPELIN-885 Add option to send email in sche...

Github user vgmartinez commented on the issue: https://github.com/apache/incubator-zeppelin/pull/918 @Leemoonsoo sorry for delay. I added the configurations in the file install.md and delete the static modifier. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-zeppelin pull request: ZEPPELIN-885 Add option to send email in sc...

Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/918 @bzz @Leemoonsoo thks for review. @bzz I'm sorry I have not explained well. I've added more description to JIRA/PR I created a new class for events outside notebook

[GitHub] incubator-zeppelin pull request: ZEPPELIN-885 Add option to send e...

Github user vgmartinez commented on the pull request: https://github.com/apache/incubator-zeppelin/pull/918#issuecomment-222380608 Hi @Leemoonsoo, I totally agree with you. I'll think a bit about this topic, if you already have some idea we can debate. thanks

[GitHub] incubator-zeppelin pull request: ZEPPELIN-885 Add option to send e...

GitHub user vgmartinez opened a pull request: https://github.com/apache/incubator-zeppelin/pull/918 ZEPPELIN-885 Add option to send email in scheduler execution ### What is this PR for? Add the option to send email when a note is runing using the scheduler ### What