[GitHub] [incubator-uniffle] zhengchenyu commented on pull request #1027: [#951] build(operator): Add the HADOOP_VERSION for Docker image.

["zhengchenyu (via GitHub)" <gi...@apache.org>]

zhengchenyu commented on PR #1027:
URL: https://github.com/apache/incubator-uniffle/pull/1027#issuecomment-1644897960

   > I'm not sure is it necessary to bind hadoop version with image. If user don't use HDFS, this is weird
   
   @zuston 
   
   Hadoop version is introduced by me in #896  
   I think we should bind hadoop version, even if we don't use HDFS. We know mapreduce, spark and tez depend on hadoop. Because the API is changed, some version of computing framework may not be compatibility with some verison of hadoop. 
   In production cluster, use the computing framework which is not  compatibility with hadoop may produce many unexpected error. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@uniffle.apache.org
For additional commands, e-mail: issues-help@uniffle.apache.org