[GitHub] [hive] TuroczyX commented on pull request #4052: HIVE-27068: Use Accumulator V2 instead of the deprecated Accumulator V1

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

TuroczyX commented on PR #4052:
URL: https://github.com/apache/hive/pull/4052#issuecomment-1495693883

   Hi @nineinfra 
   I'm not familiar with Accumulator.  (I guess it is a serialization mechanism or belongs to that topic) 
   
   I have two question with this fix:
   1. If we allow to upgrade to AC2 that old version of spark (like spark 2) can use it or it will be incompatible? What would see the user if they start to use?
   2. If we do not upgrade to AC2 would be a problem with Spark3? 
   It would be nice if it would be clear that what could be the impact of this change
   
   cc: @ayushtkn @veghlaci05 
   


-- 
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: gitbox-unsubscribe@hive.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org