Hi Jim! I also have a commit that I would like to be merged to 4.0: https://github.com/apache/impala/commit/603091ed772f3f82511fd8fec355fe9b0126933b It is a critical fix that is needed wherever the following commit also exists: https://github.com/apache/impala/commit/c65d7861d9ae28f6fc592727ff699a8155dcda2c
I have looked at the list of Jira's with target 4.0, one of them had a trivial fix, uploaded it for review: https://gerrit.cloudera.org/#/c/17412/ This is not a critical issue IMO, so I don't mind if it is not released in 4.0. On Fri, May 7, 2021 at 5:41 PM Zoltán Borók-Nagy <borokna...@apache.org> wrote: > Thank you, Jim! > > Jim Apple <apa...@jbapple.com> ezt írta (időpont: 2021. máj. 7., P 16:11): > > > Done! > > > > On Fri, May 7, 2021 at 1:42 AM Zoltán Borók-Nagy <borokna...@apache.org> > > wrote: > > > > > Hey Jim, > > > > > > My fix for IMPALA-10482 and IMPALA-10493 (the two issues are fixed in a > > > single commit) just got merged after the 4.0 branching: > > > > > > > > > https://gitbox.apache.org/repos/asf?p=impala.git;a=commit;h=f0f083e45e2c77b1499fa6fa08ff8d9dc4a2785f > > > > > > Can we add this patch to the 4.0 release? > > > > > > Thanks, > > > Zoltan > > > > > > > > > > > > On Fri, May 7, 2021 at 5:36 AM Joe McDonnell < > joemcdonn...@cloudera.com> > > > wrote: > > > > > > > I agree that it is a good idea for us to get going on the 4.0 > release. > > > > > > > > I'm working on a change to fix IMPALA-9690. I will post an updated > > review > > > > once a test run completes. If everything is looking good, I would > like > > to > > > > get this in the 4.0 release. > > > > > > > > There is also one documentation fix that I think belongs in the > Impala > > > 4.0 > > > > release: > > > > https://issues.apache.org/jira/browse/IMPALA-9770 - Remove Sentry > > > > references > > > > > > > > Thanks, > > > > Joe > > > > > > > > On Thu, May 6, 2021 at 7:55 PM Jim Apple <apa...@jbapple.com> wrote: > > > > > > > > > Sure thing! Let us know if you find owners for anything that > someone > > > > wants > > > > > to champion as a 4.0 blocker. > > > > > > > > > > On Thu, May 6, 2021 at 7:08 PM Quanlong Huang < > > huangquanl...@gmail.com > > > > > > > > > wrote: > > > > > > > > > > > Thank Jim! I planned to go through all the JIRAs target at 4.0: > > > > > > > > > > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/browse/IMPALA-10690?jql=project%20%3D%20IMPALA%20AND%20resolution%20%3D%20Unresolved%20AND%20%22Target%20Version%22%20%3D%20%22Impala%204.0%22 > > > > > > > > > > > > But due to some other urgent works and the recent holiday, my > > > progress > > > > is > > > > > > slow. Anyway, I think it's time to branch out. We've been waiting > > too > > > > > long. > > > > > > Thanks for creating the branch. > > > > > > > > > > > > Regards, > > > > > > Quanlong > > > > > > > > > > > > On Fri, May 7, 2021 at 3:29 AM Jim Apple <apa...@jbapple.com> > > wrote: > > > > > > > > > > > > > I've created a 4.0 branch (with all changes through the end of > > > April) > > > > > and > > > > > > > started testing a release candidate. Everyone who wants to > vote, > > > > > > > > > https://cwiki.apache.org/confluence/display/IMPALA/How+to+Release > > > > is a > > > > > > > guide to both releasing and voting (at the bottom). I'll let > you > > > all > > > > > know > > > > > > > when I have something to vote on. > > > > > > > > > > > > > > Thanks! > > > > > > > Jim > > > > > > > > > > > > > > On Fri, Apr 23, 2021 at 12:49 AM Csaba Ringhofer < > > > > > > csringho...@cloudera.com > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > About IMPALA-9690 (AVX support): > > > > > > > > My preferred solution would be to deprecate support for x64 > > > without > > > > > > AVX2 > > > > > > > in > > > > > > > > 4.0, but not start removing the related logic yet. > > > > > > > > - We could even add a DCHECK + flag to crash by default if no > > > AVX2 > > > > is > > > > > > > > detected, and a message that points them to Impala mailing > > > > > > lists/related > > > > > > > > Jira. > > > > > > > > - After 4.0 is released people would be free to implement > > things > > > > that > > > > > > > > assume AVX2 support on x64, for example compiling Impala/the > > > whole > > > > > > > > toolchain with -mavx2. I think that currently people are held > > > back > > > > > from > > > > > > > > doing this - why put effort in doing+benchmarking a change if > > you > > > > > can't > > > > > > > > push it to master. > > > > > > > > - If a user appears with a valid use case that needs x64 > > without > > > > AVX2 > > > > > > > then > > > > > > > > we can revise our plans, but we shouldn't ignore low hanging > > > fruits > > > > > for > > > > > > > the > > > > > > > > sake of imaginary users :) > > > > > > > > > > > > > > > > -Csaba > > > > > > > > > > > > > > > > On Sun, Apr 11, 2021 at 5:52 PM Shant Hovsepian < > > > > > > > sh...@superdupershant.com > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > +1 > > > > > > > > > > > > > > > > > > Regarding IMPALA-9690 if someone has the time to clean up > the > > > old > > > > > > > > > conditional cpu logic it would be great, but if the ARM > folks > > > > still > > > > > > > > > need some of that logic it's not critical enough to block, > > > just a > > > > > > very > > > > > > > > > nice to have? > > > > > > > > > > > > > > > > > > -Shant > > > > > > > > > > > > > > > > > > On Sun, Apr 4, 2021 at 1:09 AM Tim Armstrong < > > > > > > > tim.g.armstr...@gmail.com> > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > I agree that it would be good to get this done even if > it's > > > not > > > > > > > > perfect, > > > > > > > > > > there's so much good stuff. > > > > > > > > > > > > > > > > > > > > On Fri, 2 Apr 2021 at 17:40, Joe McDonnell < > > > > > > > joemcdonn...@cloudera.com> > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > I agree that we should wrap up Impala 4. A large amount > > of > > > > good > > > > > > > work > > > > > > > > > has > > > > > > > > > > > gone in and it belongs in a release. > > > > > > > > > > > > > > > > > > > > > > We need to decide which breaking changes are truly > > blockers > > > > for > > > > > > an > > > > > > > > > Impala 4 > > > > > > > > > > > release. My feeling is that we can't hold the release > for > > > > > > > > compatibility > > > > > > > > > > > breaking changes unless someone signs up to do those > > > changes. > > > > > > > > Breaking > > > > > > > > > > > compatibility is useful sometimes, but it can't be an > > > > > indefinite > > > > > > > hold > > > > > > > > > on > > > > > > > > > > > releases. I think a release discussion thread is a good > > way > > > > to > > > > > > > > > stimulate > > > > > > > > > > > this discussion. > > > > > > > > > > > > > > > > > > > > > > Apart from the two JIRAs you listed, one other change > > > > mentioned > > > > > > in > > > > > > > > the > > > > > > > > > > > original email is switching to use the new on-demand > > > metadata > > > > > by > > > > > > > > > default. > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > Joe > > > > > > > > > > > > > > > > > > > > > > On Fri, Apr 2, 2021 at 5:22 PM Quanlong Huang < > > > > > > > > huangquanl...@gmail.com > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > Sure. If there are no objections, I'll raise a > > discussion > > > > > > thread > > > > > > > > for > > > > > > > > > the > > > > > > > > > > > > 4.0 release. There are still some unresolved breaking > > > > > changes, > > > > > > > e.g. > > > > > > > > > > > > > > > > > > > > > > > > - IMPALA-2210: Make Parquet the default file > format > > > > > > > > > > > > - IMPALA-9690: Bump minimum x86-64 CPU > requirements > > > > > > > > > > > > > > > > > > > > > > > > Thanks > > > > > > > > > > > > Quanlong > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Mar 30, 2021 at 11:13 PM Jim Apple < > > > > > jbap...@apache.org > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > Thanks for bringing this up again, Quanlong! I > would > > > love > > > > > to > > > > > > > see > > > > > > > > a > > > > > > > > > 4.0 > > > > > > > > > > > > > release soon so that 4.1 releases and what not can > be > > > > > > prepared. > > > > > > > > > Already > > > > > > > > > > > > the > > > > > > > > > > > > > changlist from the 3.x line is going to be enormous > > and > > > > > could > > > > > > > be > > > > > > > > > hard > > > > > > > > > > > for > > > > > > > > > > > > > users to digest. > > > > > > > > > > > > > > > > > > > > > > > > > > On Mon, Mar 29, 2021 at 7:22 AM Quanlong Huang < > > > > > > > > > > > huangquanl...@gmail.com> > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > Reviving this thread. I think it's time to create > > the > > > > > 4.0.0 > > > > > > > > > branch > > > > > > > > > > > and > > > > > > > > > > > > > > prepare for the release now. Any breaking change > is > > > > > landing > > > > > > > and > > > > > > > > > we > > > > > > > > > > > > should > > > > > > > > > > > > > > wait? > > > > > > > > > > > > > > > > > > > > > > > > > > > > Thanks, > > > > > > > > > > > > > > Quanlong > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Jun 18, 2020 at 5:47 AM Tim Armstrong < > > > > > > > > > > > tarmstr...@cloudera.com > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Another thing that we're looking into is > changing > > > the > > > > > > > encoded > > > > > > > > > > > runtime > > > > > > > > > > > > > > > profile representation to be more efficient - > see > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/browse/IMPALA-9378. > > > > So > > > > > > that > > > > > > > > > might > > > > > > > > > > > be > > > > > > > > > > > > a > > > > > > > > > > > > > > > default we will try to change in Impala 4.0. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Fri, Apr 24, 2020 at 12:37 PM Tim Armstrong > < > > > > > > > > > > > > > tarmstr...@cloudera.com> > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > An addendum to this - we're also considering > > > > whether > > > > > to > > > > > > > > > increase > > > > > > > > > > > > the > > > > > > > > > > > > > > > > minimum CPU version so that we can simplify > > > > > development > > > > > > > and > > > > > > > > > focus > > > > > > > > > > > > > more > > > > > > > > > > > > > > on > > > > > > > > > > > > > > > > optimising for the most common CPUs - see > > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/browse/IMPALA-9690 > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Running on commodity hardware is important > and > > we > > > > > want > > > > > > > > > people to > > > > > > > > > > > be > > > > > > > > > > > > > > able > > > > > > > > > > > > > > > > to develop on whatever machine they have > access > > > to. > > > > > At > > > > > > > most > > > > > > > > > we're > > > > > > > > > > > > > > likely > > > > > > > > > > > > > > > > going to require AVX2 support, which has been > > > > > supported > > > > > > > by > > > > > > > > > the > > > > > > > > > > > vast > > > > > > > > > > > > > > > > majority of CPUs for the better part of a > > decade. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Apr 7, 2020 at 11:50 AM Joe > McDonnell < > > > > > > > > > > > > > > joemcdonn...@cloudera.com > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >> Impala is starting development on Impala > 4.0. > > > > There > > > > > > are > > > > > > > > many > > > > > > > > > > > > > exciting > > > > > > > > > > > > > > > >> projects underway including full support for > > > Hive > > > > > ACID > > > > > > > > > tables, > > > > > > > > > > > > > > improved > > > > > > > > > > > > > > > >> multithreading support, and fault tolerance > > > > > > > improvements, > > > > > > > > > but > > > > > > > > > > > > Impala > > > > > > > > > > > > > > 4.0 > > > > > > > > > > > > > > > >> also serves as a time to alter behavior and > > > > platform > > > > > > > > > support. > > > > > > > > > > > Here > > > > > > > > > > > > > is > > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > >> current list of notable breaking changes > that > > > are > > > > > > > planned > > > > > > > > in > > > > > > > > > > > > Impala > > > > > > > > > > > > > > 4.0: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Remove support for older operating systems: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Maintaining support for older operating > > systems > > > > > > > requires a > > > > > > > > > > > > > > considerable > > > > > > > > > > > > > > > >> amount of effort, especially with changing > > > > security > > > > > > > > > > > requirements. > > > > > > > > > > > > > Each > > > > > > > > > > > > > > > >> extra operating system consumes valuable > > effort > > > > and > > > > > > can > > > > > > > > > conflict > > > > > > > > > > > > > with > > > > > > > > > > > > > > > >> support for newer operating systems, so we > > have > > > > > > decided > > > > > > > to > > > > > > > > > > > remove > > > > > > > > > > > > > > > support > > > > > > > > > > > > > > > >> for some older operating systems. > > Specifically, > > > we > > > > > > plan > > > > > > > to > > > > > > > > > drop > > > > > > > > > > > > > > support > > > > > > > > > > > > > > > for: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> - > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Centos 6 > > > > > > > > > > > > > > > >> - > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Ubuntu 14 > > > > > > > > > > > > > > > >> - > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Debian 8 > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Each of these is approaching or past its end > > of > > > > > life. > > > > > > > > Since > > > > > > > > > > > > Centos6 > > > > > > > > > > > > > > was > > > > > > > > > > > > > > > >> the only operating system still needing > Python > > > 2.6 > > > > > and > > > > > > > > > Python > > > > > > > > > > > 2.6 > > > > > > > > > > > > is > > > > > > > > > > > > > > > long > > > > > > > > > > > > > > > >> past its end of life, Python 2.6 will no > > longer > > > be > > > > > > > > > supported. In > > > > > > > > > > > > > > > addition, > > > > > > > > > > > > > > > >> Centos 7 support will be focused on newer > > > versions > > > > > > such > > > > > > > as > > > > > > > > > 7.5 > > > > > > > > > > > and > > > > > > > > > > > > > > > above. > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Remove support for Sentry: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Over the past year and a half, Impala's > Ranger > > > > > > > > authorization > > > > > > > > > > > > > > > >> functionality has achieved parity and > > surpassed > > > > the > > > > > > > > existing > > > > > > > > > > > > Sentry > > > > > > > > > > > > > > > >> authorization functionality. Impala's Sentry > > > > support > > > > > > > > > requires > > > > > > > > > > > > > ongoing > > > > > > > > > > > > > > > >> effort to test and maintain, including > > > maintenance > > > > > on > > > > > > > > Sentry > > > > > > > > > > > > itself > > > > > > > > > > > > > to > > > > > > > > > > > > > > > >> address security updates. Given the > diminished > > > > > > activity > > > > > > > in > > > > > > > > > the > > > > > > > > > > > > > Sentry > > > > > > > > > > > > > > > >> community, there is no timeline for > addressing > > > > these > > > > > > > > > security > > > > > > > > > > > > > updates. > > > > > > > > > > > > > > > >> Unless there is a maintainer for Sentry > > support, > > > > > > Impala > > > > > > > > > plans to > > > > > > > > > > > > > focus > > > > > > > > > > > > > > > its > > > > > > > > > > > > > > > >> efforts on Ranger for its authorization > system > > > and > > > > > > drop > > > > > > > > > Sentry. > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Remove support for Impala-lzo: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Impala-lzo provides code to allow Impala to > > read > > > > the > > > > > > LZO > > > > > > > > > > > > compressed > > > > > > > > > > > > > > > >> tables. LZO is GPL licensed, which is why > this > > > > > support > > > > > > > is > > > > > > > > > not > > > > > > > > > > > > > included > > > > > > > > > > > > > > > >> directly. The Impala-lzo code interacts with > > > > > internal > > > > > > > > Impala > > > > > > > > > > > code > > > > > > > > > > > > > at a > > > > > > > > > > > > > > > >> level that is error prone and intricate. > Given > > > the > > > > > low > > > > > > > > > adoption > > > > > > > > > > > of > > > > > > > > > > > > > LZO > > > > > > > > > > > > > > > and > > > > > > > > > > > > > > > >> the other compression options available, > > Impala > > > > > plans > > > > > > to > > > > > > > > > remove > > > > > > > > > > > > > > > Impala-lzo > > > > > > > > > > > > > > > >> support along with the low level interface > it > > > > used. > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Deprecations: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> In addition, we also plan to deprecate > several > > > > > > existing > > > > > > > > > features > > > > > > > > > > > > so > > > > > > > > > > > > > > that > > > > > > > > > > > > > > > >> they can be removed in a future release. > Here > > > is a > > > > > > > summary > > > > > > > > > of > > > > > > > > > > > the > > > > > > > > > > > > > most > > > > > > > > > > > > > > > >> notable ones: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> - > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Impala will default to using on demand > > > metadata > > > > > > > > > management > > > > > > > > > > > > (i.e. > > > > > > > > > > > > > > the > > > > > > > > > > > > > > > >> local catalog implementation) as > described > > in > > > > > this > > > > > > > > > > > > documentation: > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://impala.apache.org/docs/build/html/topics/impala_metadata.html > > > > > > > > > > > > > > > >> The old metadata system is now deprecated > > and > > > > may > > > > > > be > > > > > > > > > removed > > > > > > > > > > > > in a > > > > > > > > > > > > > > > >> future release. > > > > > > > > > > > > > > > >> - > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Impala is deprecating the Beeswax client > > > > protocol > > > > > > > (i.e. > > > > > > > > > > > clients > > > > > > > > > > > > > > that > > > > > > > > > > > > > > > >> connect via beeswax_port) in favor of the > > > > > > HiveServer2 > > > > > > > > > client > > > > > > > > > > > > > > > protocol. > > > > > > > > > > > > > > > >> - > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Impala is deprecating the old > > > decimal_v2=false > > > > > > > behavior > > > > > > > > > from > > > > > > > > > > > > > Impala > > > > > > > > > > > > > > > >> 2.x. The decimal_v2 query option has > > > defaulted > > > > to > > > > > > > true > > > > > > > > > since > > > > > > > > > > > > > Impala > > > > > > > > > > > > > > > 3. > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Impala is always open to new developers, and > > we > > > > > > welcome > > > > > > > > > feedback > > > > > > > > > > > > on > > > > > > > > > > > > > > > these > > > > > > > > > > > > > > > >> plans. Further discussion of other changes > and > > > > > > > > deprecations > > > > > > > > > is > > > > > > > > > > > > > ongoing > > > > > > > > > > > > > > > at > > > > > > > > > > > > > > > >> dev@impala.apache.org. > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Thanks, > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> Joe McDonnell > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >