[GitHub] incubator-zeppelin issue #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpre...

2016-06-09 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/974 @bzz I updated license files. could you please check it again? https://github.com/apache/incubator-zeppelin/pull/974/commits/99ce452b68c757c1959b9d4b7983f7d81000256e --- If

[GitHub] incubator-zeppelin issue #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpre...

2016-06-09 Thread bzz
Github user bzz commented on the issue: https://github.com/apache/incubator-zeppelin/pull/974 Great work @jongyoul ! Looks good to me except minor things listed above --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-zeppelin issue #974: ZEPPELIN-614 Part 1. Merge PostgresqlInterpre...

2016-06-08 Thread jongyoul
Github user jongyoul commented on the issue: https://github.com/apache/incubator-zeppelin/pull/974 @tzolov Could you please test this PR with Postrgres, HAWQ and so on? I've heard that you are one of the heavy users. --- If your project is set up for it, you can reply to this email