drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hanifi Gunes" <hgu...@maprtech.com>
Subject Re: Review Request 30684: DRILL-2127: make JDBC tests re-use bit instances; reset cached bits on failure
Date Fri, 06 Feb 2015 23:02:56 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30684/
-----------------------------------------------------------

(Updated Feb. 6, 2015, 11:02 p.m.)


Review request for drill, Daniel Barclay and Venki Korukanti.


Changes
-------

uploading the most recent patch.


Bugs: DRILL-2127
    https://issues.apache.org/jira/browse/DRILL-2127


Repository: drill-git


Description
-------

DRILL-2127: make JDBC tests re-use bit instances; reset cached bits on failure


Diffs (updated)
-----

  common/src/main/java/org/apache/drill/common/config/DrillConfig.java f075cc9343652f32a467eedf2b19f836d0150537

  exec/jdbc/src/test/java/org/apache/drill/jdbc/CachingConnectionFactory.java PRE-CREATION

  exec/jdbc/src/test/java/org/apache/drill/jdbc/ConnectionFactory.java PRE-CREATION 
  exec/jdbc/src/test/java/org/apache/drill/jdbc/ConnectionInfo.java PRE-CREATION 
  exec/jdbc/src/test/java/org/apache/drill/jdbc/JdbcTest.java 3f44ed88222f33af28e95c235daca92e6955278d

  exec/jdbc/src/test/java/org/apache/drill/jdbc/MultiConnectionCachingFactory.java PRE-CREATION

  exec/jdbc/src/test/java/org/apache/drill/jdbc/NonClosableConnection.java PRE-CREATION 
  exec/jdbc/src/test/java/org/apache/drill/jdbc/SingleConnectionCachingFactory.java PRE-CREATION

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/JdbcAssert.java 49d7c1182cbce3c3ba87ae4f120f22d57624c722

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/JdbcTestQueryBase.java 64e1f029e8f051b66d3174cf2614b8a11c99109c

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestAggregateFunctionsQuery.java f6295bcefee0f480a2384eaf6a1d6fe4dc215943

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestFunctionsQuery.java 881834be082b42a7c0a4de8d3bcb8da59573f6c2

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestHiveScalarUDFs.java 705152ef0f7469323385e15bc442596e06702ada

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestHiveStorage.java 86359e04c8f56a7feb8179d0b04c1614df1ebc64

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestMetadataDDL.java c52eafd1c03662789a43981c627b6819ed2d1144

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/TestViews.java 28fa4a75bc6be2caea16c59d03e18f6c45a3ddc3


Diff: https://reviews.apache.org/r/30684/diff/


Testing
-------

The patch does affect only JDBC unit tests.

Unit tests pass.


Thanks,

Hanifi Gunes


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message