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

(Updated 2011-06-12 04:35:06.377639)


Review request for hive, John Sichi and Yongqiang He.


Changes
-------

forgot to include new unit tests, included in this patch


Summary
-------

Instead of excluding the operator from the opCtxMap entirely, I added an 
additional check to the operator initialization to make sure that at least one 
file path for the operator was valid before initialization.


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


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/MapOperator.java 96a1631 
  ql/src/test/queries/clientpositive/nestedvirtual.q PRE-CREATION 
  ql/src/test/results/clientpositive/nestedvirtual.q.out PRE-CREATION 

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


Testing
-------

Passes authorization_part.q and nestedvirtual.q. Running test suite now to 
check for any other regressions.


Thanks,

Syed

Reply via email to