[GitHub] drill pull request: Issues/drill 3791

2015-10-01 Thread aleph-zero
Github user aleph-zero commented on a diff in the pull request: https://github.com/apache/drill/pull/177#discussion_r40960796 --- Diff: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcRecordReader.java --- @@ -364,13 +368,15 @@ void copy(int index) throws

[GitHub] drill pull request: Issues/drill 3791

2015-10-01 Thread aleph-zero
Github user aleph-zero commented on a diff in the pull request: https://github.com/apache/drill/pull/177#discussion_r40960820 --- Diff: contrib/storage-jdbc/src/main/java/org/apache/drill/exec/store/jdbc/JdbcRecordReader.java --- @@ -364,13 +368,15 @@ void copy(int index) throws

[GitHub] drill pull request: Issues/drill 3791

2015-10-01 Thread aleph-zero
Github user aleph-zero commented on a diff in the pull request: https://github.com/apache/drill/pull/177#discussion_r40960677 --- Diff: contrib/storage-jdbc/pom.xml --- @@ -15,62 +15,241 @@ See the License for the specific language governing permissions and limitations

[GitHub] drill pull request: Issues/drill 3791

2015-10-01 Thread aleph-zero
Github user aleph-zero commented on a diff in the pull request: https://github.com/apache/drill/pull/177#discussion_r40960694 --- Diff: contrib/storage-jdbc/pom.xml --- @@ -15,62 +15,241 @@ See the License for the specific language governing permissions and limitations

[GitHub] drill pull request: Issues/drill 3791

2015-09-30 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/177#discussion_r40868816 --- Diff: contrib/storage-jdbc/src/test/java/org/apache/drill/exec/store/jdbc/TestJdbcPluginWithMySQLIT.java --- @@ -0,0 +1,67 @@ +/* + * Licensed

[GitHub] drill pull request: Issues/drill 3791

2015-09-30 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/177#discussion_r40868882 --- Diff: contrib/storage-jdbc/src/test/resources/mysql-test-data.sql --- @@ -0,0 +1,53 @@ + +set global time_zone = "+00:00"; --- End diff --

[GitHub] drill pull request: Issues/drill 3791

2015-09-30 Thread jacques-n
Github user jacques-n commented on a diff in the pull request: https://github.com/apache/drill/pull/177#discussion_r40868595 --- Diff: contrib/storage-jdbc/pom.xml --- @@ -15,62 +15,241 @@ See the License for the specific language governing permissions and limitations

[GitHub] drill pull request: Issues/drill 3791

2015-09-29 Thread aleph-zero
GitHub user aleph-zero opened a pull request: https://github.com/apache/drill/pull/177 Issues/drill 3791 You can merge this pull request into a Git repository by running: $ git pull https://github.com/aleph-zero/drill issues/DRILL-3791 Alternatively you can review and apply