drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Daniel Barclay" <dbarc...@maprtech.com>
Subject Re: Review Request 32695: DRILL-2632: Added assertion check to DRILL-2463 test.
Date Tue, 31 Mar 2015 21:31:57 GMT

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

(Updated March 31, 2015, 9:31 p.m.)


Review request for drill, Mehant Baid and Parth Chandra.


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


Repository: drill-git


Description
-------

In Drill2463GetNullsFailedWithAssertionsBugTest, added check that Java     assertions are
enabled, since test depends on their being enabled.


Diffs
-----

  exec/jdbc/src/test/java/org/apache/drill/jdbc/test/Drill2463GetNullsFailedWithAssertionsBugTest.java
c8346b1 

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


Testing
-------

Ran with assertions disabled to confirm detection works.

Ran existing tests.


Thanks,

Daniel Barclay


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