You are viewing a plain text version of this content. The canonical link for it is here.
Posted to gitbox@hive.apache.org by GitBox <gi...@apache.org> on 2021/01/13 10:48:02 UTC
[GitHub] [hive] pvargacl opened a new pull request #1862: HIVE-24630: clean up multiple parseDelta in AcidUtils
pvargacl opened a new pull request #1862:
URL: https://github.com/apache/hive/pull/1862
### What changes were proposed in this pull request?
Remove multiple parsedDelta implementation in AcidUtils:
- Remove code duplication
- Use ParsedDeltaLight everywhere where rawformat is not used, because parsing that is cheaper
### Why are the changes needed?
code quality
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Previous unit tests
----------------------------------------------------------------
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] deniskuzZ merged pull request #1862: HIVE-24630: clean up multiple parseDelta in AcidUtils
Posted by GitBox <gi...@apache.org>.
deniskuzZ merged pull request #1862:
URL: https://github.com/apache/hive/pull/1862
----------------------------------------------------------------
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