Skip to content

Conversation

@cwsteinbach
Copy link
Contributor

No description provided.

@jackye1995
Copy link
Contributor

Thanks for the additional patch! I read through a few Hive related classes and those are all I could find. @marton-bod @lcspinter @pvary anything else on your side that should be noted? I also saw a few bug fixes done by Marton for Tez issues.

@marton-bod
Copy link
Collaborator

Thanks @jackye1995! I think we might not want to list the Tez fixes, because those were write-related fixes and we don't yet support Tez writes - so don't want to confuse users :)

However, it might be worth including these:
Hive: enable dropping HMS tables despite metadata problems (#2583)
Hive: Add table-level JVM lock on commits (#2547)

But it's up to you, not sure what patches make the cut into the release docs. Thanks again for the call-out!

@marton-bod
Copy link
Collaborator

One more new feature to consider:
Hive: Vectorized ORC reads for Hive #2613

@cwsteinbach
Copy link
Contributor Author

@jackye1995 @marton-bod I added the following PRs:
#2565, #2583, #2547, #2613.

@jackye1995 please take a look and approve if everything looks good. Thanks!

Copy link
Contributor

@jackye1995 jackye1995 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you for the changes! @danielcweeks any other comments on your side?

@rdblue
Copy link
Contributor

rdblue commented Aug 17, 2021

I'm going to spend some time today going through commits. I got a start at it yesterday and found several things missing, including some that were caught here, so I'd like to make sure that we get it all.

@cwsteinbach cwsteinbach merged commit 335bf40 into apache:master Aug 19, 2021
@cwsteinbach cwsteinbach deleted the release-notes-redux branch August 19, 2021 19:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants