You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Thejas Nair <th...@hortonworks.com> on 2013/08/03 04:07:32 UTC

Re: Review Request 10887: HIVE-4343

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

(Updated Aug. 3, 2013, 2:07 a.m.)


Review request for hive.


Changes
-------

rebased patch


Repository: hive-git


Description
-------

HIVE-4343


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java f1e69d9 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java f38ba94 
  shims/src/0.20/java/org/apache/hadoop/hive/shims/Hadoop20Shims.java c9baa7f 
  shims/src/common-secure/java/org/apache/hadoop/hive/shims/HadoopShimsSecure.java 9edcafd 
  shims/src/common-secure/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge20S.java 5a379df 
  shims/src/common/java/org/apache/hadoop/hive/shims/HadoopShims.java b0f5077 

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


Testing
-------

 tested that it works as desired with kerberos secure cluster


Thanks,

Thejas Nair

Re: Review Request 10887: HIVE-4343

Posted by Gunther Hagleitner <gh...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10887/#review24603
-----------------------------------------------------------



ql/src/java/org/apache/hadoop/hive/ql/exec/SecureCmdDoAs.java
<https://reviews.apache.org/r/10887/#comment48640>

    Small indentation problem.



ql/src/java/org/apache/hadoop/hive/ql/exec/SecureCmdDoAs.java
<https://reviews.apache.org/r/10887/#comment48641>

    missing space


- Gunther Hagleitner


On Aug. 3, 2013, 2:32 a.m., Thejas Nair wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10887/
> -----------------------------------------------------------
> 
> (Updated Aug. 3, 2013, 2:32 a.m.)
> 
> 
> Review request for hive.
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> HIVE-4343
> 
> 
> Diffs
> -----
> 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/SecureCmdDoAs.java PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java f1e69d9 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java f38ba94 
>   shims/src/0.20/java/org/apache/hadoop/hive/shims/Hadoop20Shims.java c9baa7f 
>   shims/src/common-secure/java/org/apache/hadoop/hive/shims/HadoopShimsSecure.java 9edcafd 
>   shims/src/common-secure/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge20S.java 5a379df 
>   shims/src/common/java/org/apache/hadoop/hive/shims/HadoopShims.java b0f5077 
> 
> Diff: https://reviews.apache.org/r/10887/diff/
> 
> 
> Testing
> -------
> 
>  tested that it works as desired with kerberos secure cluster
> 
> 
> Thanks,
> 
> Thejas Nair
> 
>

Re: Review Request 10887: HIVE-4343

Posted by Thejas Nair <th...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10887/
-----------------------------------------------------------

(Updated Aug. 3, 2013, 2:32 a.m.)


Review request for hive.


Changes
-------

HIVE-4343.3.patch - adds missing file, fix compile issue after rebase


Repository: hive-git


Description
-------

HIVE-4343


Diffs (updated)
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/SecureCmdDoAs.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecDriver.java f1e69d9 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java f38ba94 
  shims/src/0.20/java/org/apache/hadoop/hive/shims/Hadoop20Shims.java c9baa7f 
  shims/src/common-secure/java/org/apache/hadoop/hive/shims/HadoopShimsSecure.java 9edcafd 
  shims/src/common-secure/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge20S.java 5a379df 
  shims/src/common/java/org/apache/hadoop/hive/shims/HadoopShims.java b0f5077 

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


Testing
-------

 tested that it works as desired with kerberos secure cluster


Thanks,

Thejas Nair