----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5227/ -----------------------------------------------------------
Review request for hive and Carl Steinbach. Summary ------- Support include/exclude macros in the test and build property to trigger inclusion or exclusion of the test based on the macros. The test macro is of the format [EX|IN]CLUDE_IF_TEST_PROP_DEFINED(property,val1,val2,...) This will cause the test to be include or excluded based on the test.include.property or test.exclude.property passed to the build command line. For example, test101.q - -- EXCLUDE_IF_TEST_PROP_DEFINED(fooProp, 100) .. ant -Dtest.exclude.property="foo:100" test This will cause the test run to exclude the test101.q script This addresses bug HIVE-3034. https://issues.apache.org/jira/browse/HIVE-3034 Diffs ----- build-common.xml b2135b3 ql/src/test/org/apache/hadoop/hive/ql/QTestUtil.java 93d002a ql/src/test/queries/clientpositive/join91.q PRE-CREATION ql/src/test/queries/clientpositive/join91.q.out PRE-CREATION ql/src/test/queries/clientpositive/join92.q PRE-CREATION ql/src/test/queries/clientpositive/join99.q PRE-CREATION ql/src/test/queries/clientpositive/join99.q.out PRE-CREATION Diff: https://reviews.apache.org/r/5227/diff Testing ------- Added dummy test that use the new macro Thanks, Prasad