[GitHub] [hive] miklosgergely opened a new pull request #1846: HIVE-24611 Remove unnecessary parameter from AbstractAlterTableOperation

[GitBox <gi...@apache.org>]

miklosgergely opened a new pull request #1846:
URL: https://github.com/apache/hive/pull/1846


   ### What changes were proposed in this pull request?
   AbstractAlterTableOperation has a private method (finalizeAlterTableWithWriteIdOp) which receives one of it's field, which doesn't make much sense.
   
   ### Why are the changes needed?
   To have a cleaner code.
   
   ### Does this PR introduce _any_ user-facing change?
   No.
   
   ### How was this patch tested?
   All the tests are still running.


----------------------------------------------------------------
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.

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

[GitHub] [hive] miklosgergely merged pull request #1846: HIVE-24611 Remove unnecessary parameter from AbstractAlterTableOperation

[GitBox <gi...@apache.org>]

miklosgergely merged pull request #1846:
URL: https://github.com/apache/hive/pull/1846


   


----------------------------------------------------------------
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.

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