[
https://issues.apache.org/jira/browse/OOZIE-734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13222708#comment-13222708
]
[email protected] commented on OOZIE-734:
-----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4176/#review5625
-----------------------------------------------------------
Ship it!
+1
- Mohammad
On 2012-03-05 17:51:33, Alejandro Abdelnur wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/4176/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-03-05 17:51:33)
bq.
bq.
bq. Review request for oozie.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. The KerberosHadoopAccessorService subclass was implemented as a separate
implementation to provide support for pre-UGI and UGI versions of Hadoop at the
same time. The DoAs/KerberosDoAs classes were there to make testcases run with
pre-UGI and UGI versions of Hadoop at the same time.
bq.
bq. As the versions of Hadoop supported by trunk (3.2.0) all have UGI, we
don't need these classes anymore.
bq.
bq. The KerberosHadoopAccessorService logic can be folded into the
HadoopAccessorService & the doAs classes can be removed.
bq.
bq.
bq. This addresses bug OOZIE-734.
bq. https://issues.apache.org/jira/browse/OOZIE-734
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. trunk/core/src/main/java/org/apache/oozie/action/hadoop/AuthHelper.java
1297142
bq. trunk/core/src/main/java/org/apache/oozie/action/hadoop/DoAs.java
1297142
bq.
trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java
1297142
bq.
trunk/core/src/main/java/org/apache/oozie/action/hadoop/KerberosAuthHelper.java
1297142
bq.
trunk/core/src/main/java/org/apache/oozie/action/hadoop/KerberosDoAs.java
1297142
bq.
trunk/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java
1297142
bq.
trunk/core/src/main/java/org/apache/oozie/service/KerberosHadoopAccessorService.java
1297142
bq. trunk/core/src/main/resources/oozie-default.xml 1297142
bq.
trunk/core/src/test/java/org/apache/oozie/action/hadoop/MainTestCase.java
1297142
bq. trunk/core/src/test/java/org/apache/oozie/action/hadoop/PigTestCase.java
1297142
bq.
trunk/core/src/test/java/org/apache/oozie/action/hadoop/ShellTestCase.java
1297142
bq.
trunk/core/src/test/java/org/apache/oozie/service/TestHadoopAccessorService.java
1297142
bq.
trunk/core/src/test/java/org/apache/oozie/service/TestKerberosHadoopAccessorService.java
1297142
bq. trunk/core/src/test/java/org/apache/oozie/test/XFsTestCase.java 1297142
bq. trunk/docs/src/site/twiki/AG_Install.twiki 1297142
bq. trunk/docs/src/site/twiki/ENG_Building.twiki 1297142
bq.
trunk/examples/src/test/java/org/apache/oozie/example/TestLocalOozieExample.java
1297142
bq.
bq. Diff: https://reviews.apache.org/r/4176/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Run all testcases successfully
bq.
bq.
bq. Thanks,
bq.
bq. Alejandro
bq.
bq.
> Simplify Kerberos/HadoopAccessorService code and remove Kerberos/DoAs code
> --------------------------------------------------------------------------
>
> Key: OOZIE-734
> URL: https://issues.apache.org/jira/browse/OOZIE-734
> Project: Oozie
> Issue Type: Improvement
> Components: core, security
> Affects Versions: 3.2.0
> Reporter: Alejandro Abdelnur
> Assignee: Alejandro Abdelnur
> Fix For: 3.2.0
>
>
> The KerberosHadoopAccessorService subclass was implemented as a separate
> implementation to provide support for pre-UGI and UGI versions of Hadoop at
> the same time. The DoAs/KerberosDoAs classes were there to make testcases run
> with pre-UGI and UGI versions of Hadoop at the same time.
> As the versions of Hadoop supported by trunk (3.2.0) all have UGI, we don't
> need these classes anymore.
> The KerberosHadoopAccessorService logic can be folded into the
> HadoopAccessorService & the doAs classes can be removed.
> This will simplify the code and avoid a common source of confusion for users.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
