Thanks Dongjoon for the details! Looks like we are going to need a follow-up release sooner than later!
On the positive side, HIVE-25386 <https://github.com/apache/hive/pull/2531> passed tests and we can now move forward. Cheers, Panagiotis On Mon, Jul 26, 2021 at 9:56 AM Dongjoon Hyun <dongj...@apache.org> wrote: > Hi, All. > > It turns out that 2.8.0 has a regression in terms of the dependency > although it passed all tests. > At Apache Hive 2.2.0, HIVE-13906 removed the Guava compile dependency. > However, recently, HIVE-24542 converts the test dependency to the compile > dependency. > > > https://github.com/apache/hive/pull/1784/files#diff-e5e45b8481f3d87f4fa561d45b22fa20bb5084e5240d825717e6bde511b733baL122 > > I noticed this breaking dependency change during Apache ORC integration > stage. > > https://github.com/apache/orc/pull/773 > > For now, I filed a JIRA issue and made a PR. Please comment your opinion > on the Hive and ORC PRs. > - HIVE-25386: hive-storage-api should not have guava compile dependency > - https://github.com/apache/hive/pull/2531 > > Bests, > Dongjoon. > > On 2021/07/23 16:40:39, "Owen O'Malley" <owen.omal...@gmail.com> wrote: > > I can do the publishing. Thank you all! > > > > .. Owen > > > > On Fri, Jul 23, 2021 at 10:16 AM Panos Garefalakis <panga...@gmail.com> > > wrote: > > > > > Thank you all for your help! With 3 binding votes this release passed! > > > I am going to send a vote result follow up but will need some PMC help > > > creating the new Storage API version (on Jira) and releasing (upload > > > artifacts)! > > > > > > Cheers, > > > Panagiotis > > > > > > On Fri, Jul 23, 2021 at 2:21 AM Chao Sun <sunc...@apache.org> wrote: > > > > > > > Thanks Owen! I just verified the checksum and gpg signature and they > both > > > > look good, so +1 too. > > > > > > > > Panos: please fix the "Fix version" of the JIRA when you get a > chance. > > > > Thanks. > > > > > > > > Best, > > > > Chao > > > > > > > > > > > > On Thu, Jul 22, 2021 at 2:29 PM Owen O'Malley < > owen.omal...@gmail.com> > > > > wrote: > > > > > > > >> Chao, > > > >> Panos key doesn't seem to have propagated to the Apache servers. > It > > > >> referenced here: > > > >> > > > >> https://people.apache.org/keys/committer/ as "pgaref > > > >> 7DFAB216AB7D96B3B2072184DC11DE4D00F8FA1D" > > > >> > > > >> The key itself can be found here: > > > >> > > > >> > > > >> > > > > https://keyserver.ubuntu.com/pks/lookup?search=pgaref&fingerprint=on&op=index > > > >> > > > >> > > > >> On Wed, Jul 21, 2021 at 8:47 PM Chao Sun <sunc...@apache.org> > wrote: > > > >> > > > >> > I built the source from the branch and ran the tests, which all > > > passed. > > > >> > However I was not able to find the public GPG key. Panos: could > you > > > >> point > > > >> > me to the location? > > > >> > > > > >> > Also seems we should create a new version 2.8.0 in the JIRA page: > > > >> > > > > >> > > > > >> > > > > https://issues.apache.org/jira/projects/HIVE?selectedItem=com.atlassian.jira.jira-projects-plugin:release-page > > > >> > and update "Fix version" of > > > >> > https://issues.apache.org/jira/browse/HIVE-24458 > > > >> > . > > > >> > > > > >> > Chao > > > >> > > > > >> > On Wed, Jul 21, 2021 at 9:10 AM Szehon Ho < > szehon.apa...@gmail.com> > > > >> wrote: > > > >> > > > > >> > > +1 (binding) > > > >> > > > > > >> > > * Built module > > > >> > > * Ran tests > > > >> > > * Checked artifact checksum and signature > > > >> > > > > > >> > > Thanks > > > >> > > Szehon > > > >> > > > > > >> > > On Tue, Jul 20, 2021 at 2:11 PM Owen O'Malley < > > > owen.omal...@gmail.com > > > >> > > > > >> > > wrote: > > > >> > > > > > >> > > > I think we should go ahead and release storage-api 2.8.0 and > catch > > > >> it > > > >> > on > > > >> > > > the next cycle. HIVE-25190 is a long standing bug that rarely > > > >> affects > > > >> > > > users. (We have had a user at LinkedIn hit it, which is why I > > > fixed > > > >> > it.) > > > >> > > > I'll sign up to make the 2.8.1 (and 2.7.3) bug fix releases > > > >> afterwards. > > > >> > > > > > > >> > > > .. Owen > > > >> > > > > > > >> > > > On Tue, Jul 20, 2021 at 8:53 PM Chao Sun <sunc...@apache.org> > > > >> wrote: > > > >> > > > > > > >> > > > > Going to check the release and vote here too. Since > HIVE-25190 > > > is > > > >> > > already > > > >> > > > > merged, instead of waiting for another release, should we > start > > > >> > another > > > >> > > > RC1 > > > >> > > > > with that included? > > > >> > > > > > > > >> > > > > Chao > > > >> > > > > > > > >> > > > > On Tue, Jul 20, 2021 at 1:30 PM Dongjoon Hyun < > > > >> dongj...@apache.org> > > > >> > > > wrote: > > > >> > > > > > > > >> > > > > > +1 > > > >> > > > > > > > > >> > > > > > * Build and tested locally. > > > >> > > > > > > > > >> > > > > > Thanks, > > > >> > > > > > Dongjoon. > > > >> > > > > > > > > >> > > > > > On 2021/07/19 23:15:46, "Owen O'Malley" < > > > owen.omal...@gmail.com > > > >> > > > > >> > > > wrote: > > > >> > > > > > > +1 (binding): > > > >> > > > > > > * Built and tested > > > >> > > > > > > * Built hive main branch using it > > > >> > > > > > > * Verified signatures and checksums > > > >> > > > > > > > > > >> > > > > > > It is too bad that we didn't get HIVE-25190 into it, but > > > that > > > >> can > > > >> > > > wait > > > >> > > > > > for > > > >> > > > > > > 2.8.1. > > > >> > > > > > > > > > >> > > > > > > .. Owen > > > >> > > > > > > > > > >> > > > > > > On Mon, Jun 28, 2021 at 9:44 PM Pavan Lanka > > > >> > > <pla...@apple.com.invalid > > > >> > > > > > > > >> > > > > > > wrote: > > > >> > > > > > > > > > >> > > > > > > > +1 (non-binding) > > > >> > > > > > > > > > > >> > > > > > > > I have done the following: > > > >> > > > > > > > * Built and Tested storage-release-2.8.0-rc0 using > > > OpenJDK8 > > > >> > > > > > > > * Built and Tested ORC with updated storage api > version > > > >> > > > > > > > - Had to fix a test class that implements > PredicateLeaf > > > >> which > > > >> > > > has a > > > >> > > > > > new > > > >> > > > > > > > method. This is a breaking change but I think this > should > > > >> be ok > > > >> > > > > > > > * Verified the performance gains of HIVE-24458 > > > >> > > > > > > > > > > >> > > > > > > > Regards, > > > >> > > > > > > > Pavan > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > On Jun 21, 2021, at 8:07 AM, Panos Garefalakis < > > > >> > > > panga...@gmail.com > > > >> > > > > > > > > >> > > > > > > > wrote: > > > >> > > > > > > > > > > > >> > > > > > > > > Hello all, > > > >> > > > > > > > > > > > >> > > > > > > > > Following on previous discussions, I would like to > > > >> propose a > > > >> > > new > > > >> > > > > > > > > storage-api release including HIVE-24458 > > > >> > > > > > > > > <https://issues.apache.org/jira/browse/HIVE-24458>. > > > >> > > > > > > > > > > > >> > > > > > > > > Shall we release the following artifacts as Hive > Storage > > > >> API > > > >> > > > 2.8.0? > > > >> > > > > > > > > > > > >> > > > > > > > > tar: > http://home.apache.org/~pgaref/hive-storage-2.8.0/ > > > >> > > > > > > > > tag: > > > >> > > > > > > > > > > >> > > > > > > > >> > > https://github.com/apache/hive/releases/tag/storage-release-2.8.0-rc0 > > > >> > > > > > > > > jiras: > > > >> > > > > > > > > https://issues.apache.org/jira/projects/HIVE/versions/12350287 > > > >> > > > > > > > > > > > >> > > > > > > > > Cheers, > > > >> > > > > > > > > Panagiotis > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > >> > > > > > > > >> > > > > > > >> > > > > > >> > > > > >> > > > > > > > > > >
