Henry Robinson has posted comments on this change.
Change subject: IMPALA-4041: Limit catalog and admission control updates to
coordinators
..
Patch Set 5: Code-Review+2
(6 comments)
http://gerrit.cloudera.org:8080/#/c/6344/5
Dan Hecht has posted comments on this change.
Change subject: IMPALA-3203: Part 2: per-core free lists in buffer pool
..
Patch Set 13:
(17 comments)
mostly just comments about the headers so far. also, please rebase on the
ev
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
..
Patch Set 8:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
..
Patch Set 8: Code-Review+1
Rebase and fix a test-compilation bug.
--
To view, visit http
Hello Impala Public Jenkins, Jim Apple, Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5688
to look at the new patch set (#8).
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
...
Henry Robinson has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6497
Change subject: Fix missing declaration of std::max in hdfs-avro-scanner-test
..
Fix missing declaration of std::max in hdfs-avro-scanne
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4623: Thread level file handle caching
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6478/1/be/src/runtime/disk-io-mgr.h
File be/src/ru
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4701: make distcc work reliably with clang
..
Patch Set 5: Code-Review+1
Fixed one more bug discovered by running a test jenkins job
--
To view, visit h
Hello Lars Volker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6493
to look at the new patch set (#5).
Change subject: IMPALA-4701: make distcc work reliably with clang
..
IMPALA-4701:
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-4623: Thread level file handle caching
..
Patch Set 1:
(20 comments)
http://gerrit.cloudera.org:8080/#/c/6478/1/be/src/runtime/disk-io-mgr.cc
File be/
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
..
Patch Set 7: Verified-1
Build failed: http://jenkins.impala.io:8080/job/gerrit-veri
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
Patch Set 7: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/6469
To unsubscribe, visit http:/
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
IMPALA-5113: fix dirty unpinned invariant
There were two bugs:
* The invariant was too strict and d
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
..
Patch Set 7:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-5110: Add deb support to dump_breakpad_symbols.py
..
IMPALA-5110: Add deb support to dump_breakpad_symbols.py
Change-Id: I524d2fe4660551c
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5110: Add deb support to dump_breakpad_symbols.py
..
Patch Set 2: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/6462
To unsubscrib
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-4673: Use --local_library_dir for tzdb startup scratch
space
..
IMPALA-4673: Use --local_library_dir for tzdb startup scratch space
Curr
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4673: Use --local_library_dir for tzdb startup scratch
space
..
Patch Set 7: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5523
T
Jim Apple has posted comments on this change.
Change subject: IMPALA-4758: (1/2) Update gutil/ from Kudu@a1bfd7b
..
Patch Set 6:
Also fixes IMPALA-5116
--
To view, visit http://gerrit.cloudera.org:8080/5687
To unsubscribe, vi
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4701: make distcc work reliably with clang
..
Patch Set 4: Code-Review+1
Fixed a couple of bugs - when I initially tested I didn't do so on a completely
Hello Lars Volker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6493
to look at the new patch set (#4).
Change subject: IMPALA-4701: make distcc work reliably with clang
..
IMPALA-4701:
Jim Apple has posted comments on this change.
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
..
Patch Set 7: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/5688
To unsubscri
Hello Lars Volker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6493
to look at the new patch set (#3).
Change subject: IMPALA-4701: make distcc work reliably with clang
..
IMPALA-4701:
Tim Armstrong has uploaded a new patch set (#13).
Change subject: IMPALA-3203: Part 2: per-core free lists in buffer pool
..
IMPALA-3203: Part 2: per-core free lists in buffer pool
Add per-core lists of clean pages and free page
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3203: Part 2: per-core free lists in buffer pool
..
Patch Set 13:
Fix bugs with the low water mark tracking.
--
To view, visit http://gerrit.cloudera.or
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3203: Part 2: per-core free lists in buffer pool
..
Patch Set 12:
Rebased on https://gerrit.cloudera.org/#/c/6469/4
--
To view, visit http://gerrit.clou
Tim Armstrong has uploaded a new patch set (#12).
Change subject: IMPALA-3203: Part 2: per-core free lists in buffer pool
..
IMPALA-3203: Part 2: per-core free lists in buffer pool
Add per-core lists of clean pages and free page
Lars Volker has posted comments on this change.
Change subject: IMPALA-4166: Add SORT BY sql clause
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6495/1/fe/src/main/java/org/apache/impala/analysis/TableProper
Lars Volker has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6495
Change subject: IMPALA-4166: Add SORT BY sql clause
..
IMPALA-4166: Add SORT BY sql clause
This change adds support for adding SORT BY (..
Hello Jim Apple, Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5688
to look at the new patch set (#7).
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
..
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4758: (2/2) Impala-side changes to build with latest
gutil
..
Patch Set 6:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/5688/6/be/src/common/atomic
Hello Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5687
to look at the new patch set (#6).
Change subject: IMPALA-4758: (1/2) Update gutil/ from Kudu@a1bfd7b
..
IMPALA-47
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
Patch Set 7:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/415/
--
To view, visit
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
Patch Set 5:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/6469/5/be/src/runtime/bufferpool/buffer-pool-internal
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
Patch Set 7: Code-Review+2
Carry +2
--
To view, visit http://gerrit.cloudera.org:8080/6469
To unsubscribe, visit h
Hello Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6469
to look at the new patch set (#6).
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
IMPALA-5113: fix dirty
Lars Volker has posted comments on this change.
Change subject: IMPALA-4701: make distcc work reliably with clang
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.cloudera.org:8080/6493
To unsubscribe, visit http:
Dan Hecht has posted comments on this change.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
Patch Set 5: Code-Review+2
(3 comments)
http://gerrit.cloudera.org:8080/#/c/6469/5/be/src/runtime/bufferpool/buffer-poo
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4701: make distcc work reliably with clang
..
Patch Set 1:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/6493/1/bin/distcc/distcc.sh
File bin/distcc/d
Tim Armstrong has uploaded a new patch set (#2).
Change subject: IMPALA-4701: make distcc work reliably with clang
..
IMPALA-4701: make distcc work reliably with clang
The previous compiler-switching method in distcc had some pr
Tim Armstrong has uploaded a new patch set (#5).
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
IMPALA-5113: fix dirty unpinned invariant
There were two bugs:
* The invariant was too strict and didn't take into acc
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
Patch Set 4:
(8 comments)
http://gerrit.cloudera.org:8080/#/c/6469/4/be/src/runtime/bufferpool/buffer-pool-internal
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4623: Thread level file handle caching
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6478/1/be/src/runtime/disk-io-mgr.h
File be/src/ru
Jim Apple has posted comments on this change.
Change subject: IMPALA-5031: Remove undefined behavior: left shift of negative
values
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6491/1/be/src/exec/read-write
Dan Hecht has posted comments on this change.
Change subject: IMPALA-5031: Remove undefined behavior: left shift of negative
values
..
Patch Set 1: Code-Review+1
Looks good, other than Tim's comments.
--
To view, visit http:
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-5110: Add deb support to dump_breakpad_symbols.py
..
Patch Set 2:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/414/
--
Lars Volker has posted comments on this change.
Change subject: IMPALA-4701: make distcc work reliably with clang
..
Patch Set 1:
(4 comments)
Thank you for fixing this! I couldn't quite figure out how the variables in
distcc
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5110: Add deb support to dump_breakpad_symbols.py
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6462
To unsubscribe, vi
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-5031: Remove undefined behavior: left shift of negative
values
..
Patch Set 1:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/6491/1/be/src/exec/read-
Dan Hecht has posted comments on this change.
Change subject: IMPALA-5113: fix dirty unpinned invariant
..
Patch Set 4:
(8 comments)
Nice cleanup.
http://gerrit.cloudera.org:8080/#/c/6469/4/be/src/runtime/bufferpool/buffer-po
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4673: Use --local_library_dir for tzdb startup scratch
space
..
Patch Set 7:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryru
Bharath Vissapragada has posted comments on this change.
Change subject: IMPALA-4673: Use --local_library_dir for tzdb startup scratch
space
..
Patch Set 7: Code-Review+2
Rebased, carrying +2. Thanks for the review Tim.
--
T
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4673: Use --local_library_dir for tzdb startup scratch
space
..
Patch Set 6: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5523
To uns
Tim Armstrong has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6493
Change subject: IMPALA-4701: make distcc work reliably with clang
..
IMPALA-4701: make distcc work reliably with clang
The previous comp
Sailesh Mukil has abandoned this change.
Change subject: IMPALA-4456: Change query_exec_state_lock_ to a reader-writer
lock
..
Abandoned
Need to think of a different strategy regarding this. See JIRA for more
information.
--
Sailesh Mukil has abandoned this change.
Change subject: IMPALA-3977: TransmitData() should not block
..
Abandoned
Addressed by one of the KuduRPC patches.
--
To view, visit http://gerrit.cloudera.org:8080/5491
To unsubscribe
Sailesh Mukil has abandoned this change.
Change subject: IMPALA-4459: Consider making ReportExecStatus() RPC execute
asynchronously
..
Abandoned
This is addressed by KuduRPC.
--
To view, visit http://gerrit.cloudera.org:8080
Lars Volker has posted comments on this change.
Change subject: IMPALA-4226, IMPALA-4227: bump max threads, handle dwz
compressed symbols
..
Patch Set 3:
Thanks for the reviews. I started a private verification build (should'v
Bharath Vissapragada has posted comments on this change.
Change subject: IMPALA-4673: Use --local_library_dir for tzdb startup scratch
space
..
Patch Set 6:
Tim, any more comments on this? Can I carry your earlier +2?
--
To
Bharath Vissapragada has posted comments on this change.
Change subject: IMPALA-4226, IMPALA-4227: bump max threads, handle dwz
compressed symbols
..
Patch Set 3:
No more comments TM. LGTM.
--
To view, visit http://gerrit.cl
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4226, IMPALA-4227: bump max threads, handle dwz
compressed symbols
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6461
Lars Volker has posted comments on this change.
Change subject: IMPALA-4226, IMPALA-4227: bump max threads, handle dwz
compressed symbols
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6461/2//COMMIT_MSG
Comm
Hello Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6461
to look at the new patch set (#3).
Change subject: IMPALA-4226, IMPALA-4227: bump max threads, handle dwz
compressed symbols
.
Lars Volker has posted comments on this change.
Change subject: IMPALA-3224: Migrate non-docs JIRA URLs
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6487/1/fe/src/main/java/org/apache/impala/analysis/CreateT
Valencia Edna Serrao has posted comments on this change.
Change subject: Ported native-toolchain to work on ppc64le
..
Patch Set 1:
@Matthew, @Tim: Thanks for reviewing the patch. I'm working on the pointers you
have suggested
65 matches
Mail list logo