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

(Updated 2011-08-08 22:42:22.232604)


Review request for hive.


Changes
-------

* Removed star imports in unit test.
* Added test case to integration test.  Call to use default NPEs before patch, 
succeeds afterwards
* Removed no-op from cli/build.xml to ensure test runs
* Added mockito into cli's ivy.xml.  The top-level ivy.xml doesn't need changed 
since the ivy settings aren't inherited.
* Didn't change the split call to tokenize, since it was as is in the original 
patch.  No need to introduce unnecessary changes. 


Summary
-------

Commands that don't return a schema cause NPE when print headers is on.


This addresses bug HIVE-2334.
    https://issues.apache.org/jira/browse/HIVE-2334


Diffs (updated)
-----

  cli/build.xml c174b22 
  cli/ivy.xml f272b91 
  cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java 9fa7bc6 
  cli/src/test/org/apache/hadoop/hive/cli/TestCliDriverMethods.java 
PRE-CREATION 
  ivy/libraries.properties af856bd 
  ql/src/test/queries/clientpositive/print_header.q 96380ab 
  ql/src/test/results/clientpositive/print_header.q.out e3946d7 

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


Testing
-------

New unit tests (both positive and negative) and verification on manual cluster.


Thanks,

Jakob

Reply via email to