[GitHub] zeppelin issue #1776: [ZEPPELIN-1824] Add MetaData exploration to JDBC Inter...

2016-12-20 Thread pmccaffrey6
Github user pmccaffrey6 commented on the issue: https://github.com/apache/zeppelin/pull/1776 Hey @bzz, Great catch! Sorry about that. I removed the unnecessary import. The error seems unrelated. It comes from: ``` 1470 21:03:56,259 ERROR

[GitHub] zeppelin issue #1776: [ZEPPELIN-1824] Add MetaData exploration to JDBC Inter...

2016-12-18 Thread pmccaffrey6
Github user pmccaffrey6 commented on the issue: https://github.com/apache/zeppelin/pull/1776 I went ahead and changed parsing of the "explore" keyword to use `split(" +")` instead of substring so that it wasn't tied to any particular word length etc.. and t

[GitHub] zeppelin issue #1776: [ZEPPELIN-1824] Add MetaData exploration to JDBC Inter...

2016-12-16 Thread pmccaffrey6
Github user pmccaffrey6 commented on the issue: https://github.com/apache/zeppelin/pull/1776 The failed test looks to be from `testSparkSQLInterpreter` from `LivyInterpreterIT.java`. More specifically, it comes from line 227 (and 226) in that file: ``` InterpreterResult

[GitHub] zeppelin issue #1776: [ZEPPELIN-1824] Add MetaData exploration to JDBC Inter...

2016-12-16 Thread pmccaffrey6
Github user pmccaffrey6 commented on the issue: https://github.com/apache/zeppelin/pull/1776 Hey guys, Thanks so much for the review! I removed the unused variable (great catch). As for the other issues, I think they make a ton of sense. I wrote the `getMetaData

[GitHub] zeppelin pull request #1776: [ZEPPELIN-1824] Add MetaData exploration to JDB...

2016-12-16 Thread pmccaffrey6
GitHub user pmccaffrey6 opened a pull request: https://github.com/apache/zeppelin/pull/1776 [ZEPPELIN-1824] Add MetaData exploration to JDBC Interpreter ### What is this PR for? Zeppelin currently has little functionality for data source exploration. This PR exists to build a

[GitHub] zeppelin issue #1414: filter applied to graph

2016-10-25 Thread pmccaffrey6
Github user pmccaffrey6 commented on the issue: https://github.com/apache/zeppelin/pull/1414 This seems like a really cool idea and something that our group, for one, would love --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as