[ 
https://issues.apache.org/jira/browse/PIG-4923?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15745833#comment-15745833
 ] 

Rohini Palaniswamy commented on PIG-4923:
-----------------------------------------

bq. I think considering what Hadoop3 will need it might not be a bad idea to 
remove the additional complexity of using shims.
   I think it should be ok for now to just leave HadoopShims in the shims 
directory. It is not too much of a complexity. It is just a class and if not 
required we can simply rename it to HadoopUtils and move it to main code base.


bq. 3) Try to get rid of reflections in code as much as possible which were 
done for Hadoop 1.x or Hbase 94. For eg: UserGroupInformation methods in 
hadoop, TableMapReduceUtil in hbase are called via reflection in HBaseStorage
You had missed the above comment in the review. Can you please look for classes 
with java.lang.reflect.Method and remove the reflection code as much as 
possible. Except for org.apache.hadoop.ipc.CallerContext which is only 
available from Hadoop 2.8, most others could be removed. 


> Drop Hadoop 1.x support in Pig 0.17
> -----------------------------------
>
>                 Key: PIG-4923
>                 URL: https://issues.apache.org/jira/browse/PIG-4923
>             Project: Pig
>          Issue Type: Improvement
>            Reporter: Daniel Dai
>            Assignee: Adam Szita
>             Fix For: 0.17.0
>
>         Attachments: PIG-4923.1.patch, PIG-4923.2.IncrementalHadoop3.patch, 
> PIG-4923.2.patch, PIG-4923.3.patch, ant-contrib-1.0b3.jar
>
>
> To facilitate the future development, we want to get rid of the legacy Hadoop 
> 1.x support and reduce the code complexity.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to