Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 22: Verified+1
--
To view, visit http://gerrit.clo
Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 22:
Build started: https://jenkins.impala.io/job/ge
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 22: Code-Review+2
Carry +2
--
To view, visit http://gerrit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 22: Verified-1
Build failed: https://jenkins.impala
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 22:
Build started: https://jenkins.impala.io/job/ge
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 22: Code-Review+2
--
To view, visit http://gerrit.
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 21: Verified-1
Build failed: https://jenkins.impala
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 21: Code-Review+2
Carry +2
--
To view, visit http://gerrit
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 21:
Build started: https://jenkins.impala.io/job/ge
Attila Jeges has uploaded a new patch set (#21). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 20:
clang-tidy job failed. Investigating.
--
To view, visi
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 20: Verified-1
Build failed: https://jenkins.impala
Impala Public Jenkins has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 20:
Build started: https://jenkins.impala.io/job/ge
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 20: Code-Review+2
Exhaustive, ASAN tests passed. Rebased the
Attila Jeges has uploaded a new patch set (#20). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 19:
Thanks for running those tests!
--
To view, visit htt
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 19: Code-Review+2
--
To view, visit http://gerrit.cloudera
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 19:
> Are we ready to go ahead and merge? Would be good to r
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 19:
Are we ready to go ahead and merge? Would be good to ru
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/9986/1
Attila Jeges has uploaded a new patch set (#19). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/9986/1
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 18: Code-Review+1
(2 comments)
http://gerrit.cloudera.org:
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 18: Code-Review+2
(1 comment)
http://gerrit.cloudera.org
Attila Jeges has uploaded a new patch set (#18). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 17:
> Uploaded patch set 17: Commit message was updated.
Ad
Attila Jeges has uploaded a new patch set (#17). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has uploaded a new patch set (#16). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has uploaded a new patch set (#15). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 14: Code-Review+2
(5 comments)
Thanks for thinking about
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 14:
> Uploaded patch set 14.
Added one more BE test for ext
Attila Jeges has uploaded a new patch set (#14). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has uploaded a new patch set (#13). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 11:
(2 comments)
Added some tests for extracting files from
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 11:
(2 comments)
Thanks for adding zip support!
We shoul
Attila Jeges has uploaded a new patch set (#12). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 11:
> (1 comment)
>
> Yeah I agree with Phil that doing JN
Attila Jeges has uploaded a new patch set (#11). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/9986
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
(1 comment)
Yeah I agree with Phil that doing JNI to j
Philip Zeyliger has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
(5 comments)
> > > > > > Uploaded patch set 9.
> >
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
> > > > > Uploaded patch set 9.
> > > >
> > > > Patch
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
> > > > Uploaded patch set 9.
> > >
> > > Patch -set 9
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 10:
I do not see any more low hanging fruits for performa
Attila Jeges has uploaded a new patch set (#10). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 9:
> > > Uploaded patch set 9.
> >
> > Patch -set 9 con
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 9:
> > Uploaded patch set 9.
>
> Patch -set 9 contains the
Attila Jeges has uploaded a new patch set (#9). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses two
Attila Jeges has uploaded a new patch set (#8). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses two
Tim Armstrong has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 7:
(15 comments)
This is going to be a big improvement. Di
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 7: Code-Review+1
(4 comments)
Thanks Attila for addressing
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 7: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 7:
(2 comments)
I focused mostly on the non-timestamp/timezone
Attila Jeges has uploaded a new patch set (#7). (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses two
Dan Hecht has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 5:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/9986/5/be/
Hello Gabor Kaszab, Zoltan Borok-Nagy, Csaba Ringhofer,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9986
to look at the new patch set (#6).
Change subject: IMPALA-3307: Add support for IANA time-zone db
...
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 5:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/9986/
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 4:
(18 comments)
http://gerrit.cloudera.org:8080/#/c/9986/4
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 5:
(19 comments)
Thanks for dealing with my previous commen
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 4:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/9986/4/
Hello Gabor Kaszab, Zoltan Borok-Nagy, Csaba Ringhofer,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9986
to look at the new patch set (#5).
Change subject: IMPALA-3307: Add support for IANA time-zone db
...
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 4: Code-Review+1
(6 comments)
Few nits otherwise looks g
Hello Gabor Kaszab, Zoltan Borok-Nagy, Csaba Ringhofer,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9986
to look at the new patch set (#4).
Change subject: IMPALA-3307: Add support for IANA time-zone db
...
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 3:
(10 comments)
http://gerrit.cloudera.org:8080/#/c/9986/3
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 3:
(10 comments)
http://gerrit.cloudera.org:8080/#/c/998
Hello Gabor Kaszab, Zoltan Borok-Nagy, Csaba Ringhofer,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9986
to look at the new patch set (#3).
Change subject: IMPALA-3307: Add support for IANA time-zone db
...
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 2:
(55 comments)
http://gerrit.cloudera.org:8080/#/c/9986/2
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 2:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/9986/2/
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 2:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/9986/2/
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 2:
(29 comments)
Dumping another batch of comments :)
http
Zoltan Borok-Nagy has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 2:
(8 comments)
http://gerrit.cloudera.org:8080/#/c/99
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 2:
(12 comments)
http://gerrit.cloudera.org:8080/#/c/998
Hello Gabor Kaszab, Zoltan Borok-Nagy, Csaba Ringhofer,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/9986
to look at the new patch set (#2).
Change subject: IMPALA-3307: Add support for IANA time-zone db
...
Attila Jeges has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 1:
(31 comments)
http://gerrit.cloudera.org:8080/#/c/9986/1
Gabor Kaszab has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 1:
(7 comments)
Spent only a limited amount of time on this
Csaba Ringhofer has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 1:
(25 comments)
http://gerrit.cloudera.org:8080/#/c/998
Zoltan Borok-Nagy has posted comments on this change. (
http://gerrit.cloudera.org:8080/9986 )
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
Patch Set 1:
(2 comments)
Just skimmed through. Will do several
Attila Jeges has uploaded this change for review. (
http://gerrit.cloudera.org:8080/9986
Change subject: IMPALA-3307: Add support for IANA time-zone db
..
IMPALA-3307: Add support for IANA time-zone db
Impala currently uses tw
79 matches
Mail list logo