Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4652: Add crcutil to build
..
Patch Set 2: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5660
To unsubscribe, visit http://gerrit.
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4651: Add LibEv to build
..
Patch Set 2:
Build failed: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/178/
--
To view, visit http://gerr
Lars Volker has posted comments on this change.
Change subject: IMPALA-2522: Add doc for sortby() and clustered hints
..
Patch Set 4:
(10 comments)
Thanks for the change. I added comments inline.
http://gerrit.cloudera.org:80
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4722: Disable log caching in test_scratch_disk
..
Patch Set 2:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/179/
--
To
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
Patch Set 3:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/5631/3/fe/src/main/java/org/apache/im
Tim Armstrong has posted comments on this change.
Change subject: Release note updates for Impala 2.8
..
Patch Set 3:
Is this a duplicate of https://gerrit.cloudera.org/#/c/5668/?
--
To view, visit http://gerrit.cloudera.org:
Jim Apple has posted comments on this change.
Change subject: IMPALA-4747: macros should only evaluate their arguments once
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5686/2/be/src/testutil/gtest-util.h
Fi
Hello Impala Public Jenkins, Jim Apple, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4878
to look at the new patch set (#6).
Change subject: IMPALA-2615: warn if Status is ignored
...
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 6: Code-Review+2
Missed one clang-tidy warning. Really need to figure out how to run it locally.
--
To view
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 6:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/180/
--
To view, visit htt
Jim Apple has posted comments on this change.
Change subject: IMPALA-4355: random query generator: modify statement execution
flow to support DML
..
Patch Set 6: Code-Review+2
carry taras's +2
--
To view, visit http://gerrit
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4355: random query generator: modify statement execution
flow to support DML
..
Patch Set 6:
Build started: http://jenkins.impala.io:8080/job/ger
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 6:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4878/6/be/src/runtime/disk-io-mgr-test.cc
File be/src/run
Dan Hecht has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 6:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4878/6/be/src/runtime/disk-io-mgr-test.cc
File be/src/runtime
Dan Hecht has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 6:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/4878/6/be/src/runtime/disk-io-mgr-test.cc
File be/src/runtime
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-4752: make ObjectPool more efficient
..
Patch Set 3: Code-Review+1
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5666/3/be/src/common/object-p
Hello Impala Public Jenkins, Jim Apple, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4878
to look at the new patch set (#7).
Change subject: IMPALA-2615: warn if Status is ignored
...
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4651: Add LibEv to build
..
Patch Set 3:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/182/
--
To view, visit http://ger
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 7:
Turns out there's a gcc bug that prevents this from working sometimes
(including for DiskIoMgr::Init()).
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-4722: Disable log caching in test_scratch_disk
..
IMPALA-4722: Disable log caching in test_scratch_disk
test_scratch_disk fails sporadica
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4722: Disable log caching in test_scratch_disk
..
Patch Set 2: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5669
To unsubscribe,
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-4748: crash in TmpFileMgr when hitting process mem limit
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5683/2/be/src/runtime/t
Attila Jeges has abandoned this change.
Change subject: IMPALA-4216: Test became flaky:
TestTpchMemLimitError.test_low_mem_limit_q20
..
Abandoned
--
To view, visit http://gerrit.cloudera.org:8080/4572
To unsubscribe, visit ht
Hello Marcel Kornacker,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5585
to look at the new patch set (#8).
Change subject: IMPALA-1861: Simplify conditionals with constant conditions
.
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-1861: Simplify conditionals with constant conditions
..
Patch Set 8:
(4 comments)
http://gerrit.cloudera.org:8080/#/c/5585/7/fe/src/main/java/or
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4036: invalid SQL generated for partitioned table with
comment
..
Patch Set 4: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5648
To
Henry Robinson has submitted this change and it was merged.
Change subject: IMPALA-4036: invalid SQL generated for partitioned table with
comment
..
IMPALA-4036: invalid SQL generated for partitioned table with comment
For a t
Dan Hecht has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 7:
> Turns out there's a gcc bug that prevents this from working
> sometimes (including for DiskIoMgr::Init()).
Hello Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5631
to look at the new patch set (#4).
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
IMPAL
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
Patch Set 4:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/5631/3/fe/src/main/java/org/apa
Dan Hecht has posted comments on this change.
Change subject: IMPALA-4752: make ObjectPool more efficient
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5666
To unsubscribe, visit http://gerrit
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5631/4/fe/src/main/java/org/apache/imp
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4355: random query generator: modify statement execution
flow to support DML
..
Patch Set 6: Verified+1
--
To view, visit http://gerrit.cloudera
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-4355: random query generator: modify statement execution
flow to support DML
..
IMPALA-4355: random query generator: modify statement exe
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4351,IMPALA-4353: [qgen] randomly generate INSERT
statements
..
Patch Set 7:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryru
Jim Apple has posted comments on this change.
Change subject: IMPALA-4351,IMPALA-4353: [qgen] randomly generate INSERT
statements
..
Patch Set 7: Code-Review+2
carry +2
--
To view, visit http://gerrit.cloudera.org:8080/5486
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4651: Add LibEv to build
..
Patch Set 3: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5659
To unsubscribe, visit http://gerrit.cl
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4351,IMPALA-4353: [qgen] randomly generate INSERT
statements
..
Patch Set 7:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryru
zams...@cloudera.com has posted comments on this change.
Change subject: IMPALA-4739: ExprRewriter fails on HAVING clauses
..
Patch Set 6:
(2 comments)
I realize this is already merged - simply learning how to use gerrit and I
Hello Tim Armstrong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5631
to look at the new patch set (#5).
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
IMPAL
Thomas Tauber-Marshall has posted comments on this change.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
Patch Set 5:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5631/4/fe/src/main/java/org/apac
Laurel Hale has posted comments on this change.
Change subject: IMPALA-4643: Centralize tags in separate .ditamap
..
Patch Set 2: Code-Review+1
I pulled the files into my own branch and built them with the dita ot 2.3.3
into
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 7:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/185/
--
To view, visit htt
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 7: Code-Review+2
Carry +2
--
To view, visit http://gerrit.cloudera.org:8080/4878
To unsubscribe, visit http
Tim Armstrong has uploaded a new patch set (#5).
Change subject: IMPALA-4678: move query MemTracker into QueryState
..
IMPALA-4678: move query MemTracker into QueryState
The query MemTracker for query execution is now owned dire
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4678: move query MemTracker into QueryState
..
Patch Set 4:
(18 comments)
http://gerrit.cloudera.org:8080/#/c/5630/4/be/src/runtime/coordinator.cc
File b
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4752: make ObjectPool more efficient
..
Patch Set 3:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5666/3/be/src/common/object-pool.h
File be/src/commo
Hello Lars Volker, Thomas Tauber-Marshall, Dan Hecht,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5666
to look at the new patch set (#4).
Change subject: IMPALA-4752: make ObjectPool more efficient
...
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4752: make ObjectPool more efficient
..
Patch Set 4: Code-Review+2
Carry +2
--
To view, visit http://gerrit.cloudera.org:8080/5666
To unsubscribe, visit
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4752: make ObjectPool more efficient
..
Patch Set 4:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/186/
--
To view, visi
Tim Armstrong has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5702
Change subject: IMPALA-4757: addendum: avoid double underscore prefix
..
IMPALA-4757: addendum: avoid double underscore prefix
Names sta
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4747: macros should only evaluate their arguments once
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5686/2/be/src/testutil/gtest-util.
Henry Robinson has submitted this change and it was merged.
Change subject: IMPALA-4651: Add LibEv to build
..
IMPALA-4651: Add LibEv to build
Add libev 4.20 to the Impala build. This is a dependency of KRPC.
FindLibEv.cmake w
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4651: Add LibEv to build
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5659
To unsubscribe, visit http://gerrit.cloude
Henry Robinson has submitted this change and it was merged.
Change subject: IMPALA-4652: Add crcutil to build
..
IMPALA-4652: Add crcutil to build
Add crcutil, built from a git hash since there are no released versions,
to Impa
Henry Robinson has posted comments on this change.
Change subject: IMPALA-4652: Add crcutil to build
..
Patch Set 3: Code-Review+2 Verified+1
Rebase
--
To view, visit http://gerrit.cloudera.org:8080/5660
To unsubscribe, visit
Laurel Hale has posted comments on this change.
Change subject: IMPALA-4643: Centralize tags in separate .ditamap
..
Patch Set 2:
you do have a merge conflict that must be resolved.
--
To view, visit http://gerrit.cloudera.o
Jim Apple has posted comments on this change.
Change subject: IMPALA-4757: addendum: avoid double underscore prefix
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5702/1//COMMIT_MSG
Commit Message:
PS1, Line
Marcel Kornacker has posted comments on this change.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
Patch Set 5: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5631
To unsubscribe, v
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
Patch Set 5:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/187/
--
To
Tim Armstrong has uploaded a new patch set (#4).
Change subject: IMPALA-4549: consistently treat as upper bound for
timestamp year
..
IMPALA-4549: consistently treat as upper bound for timestamp year
Previously Impala
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4549: consistently treat as upper bound for
timestamp year
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5665/3/be/src/exprs/expr
Tim Armstrong has uploaded a new patch set (#2).
Change subject: IMPALA-4757: addendum: avoid double underscore prefix
..
IMPALA-4757: addendum: avoid double underscore prefix
Names starting with double underscore are implementa
Tim Armstrong has uploaded a new patch set (#3).
Change subject: IMPALA-4757: addendum: avoid double underscore in name
..
IMPALA-4757: addendum: avoid double underscore in name
Names containing double underscore are implementat
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4757: addendum: avoid double underscore in name
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5702/1//COMMIT_MSG
Commit Message:
PS1,
Tim Armstrong has uploaded a new patch set (#3).
Change subject: IMPALA-4748: crash in TmpFileMgr when hitting process mem limit
..
IMPALA-4748: crash in TmpFileMgr when hitting process mem limit
The bug is that FileGroup didn't
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-4748: crash in TmpFileMgr when hitting process mem limit
..
Patch Set 2:
(1 comment)
Hopefully that is a little clearer.
http://gerrit.cloudera.org:8080
Jim Apple has posted comments on this change.
Change subject: IMPALA-4757: addendum: avoid double underscore in name
..
Patch Set 3: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/5702
To unsubscribe, visit ht
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4757: addendum: avoid double underscore in name
..
Patch Set 3:
Build started: http://jenkins.impala.io:8080/job/gerrit-verify-dryrun/188/
--
To
Tim Armstrong has posted comments on this change.
Change subject: IMPALA-3202: implement spill-to-disk in new buffer pool
..
Patch Set 7:
(15 comments)
http://gerrit.cloudera.org:8080/#/c/5584/7/be/src/runtime/bufferpool/buffe
Tim Armstrong has uploaded a new patch set (#8).
Change subject: IMPALA-3202: implement spill-to-disk in new buffer pool
..
IMPALA-3202: implement spill-to-disk in new buffer pool
See https://goo.gl/0zuy97 for a high-level summa
Laurel Hale has posted comments on this change.
Change subject: Clean up tags for 2 query options
..
Patch Set 1: Code-Review+1
pulled and built the changes in a new branch--no merge conflicts. HTML nav
titles look good.
--
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4351,IMPALA-4353: [qgen] randomly generate INSERT
statements
..
Patch Set 7: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5486
T
Tim Armstrong has uploaded a new patch set (#9).
Change subject: IMPALA-3202: implement spill-to-disk in new buffer pool
..
IMPALA-3202: implement spill-to-disk in new buffer pool
See https://goo.gl/0zuy97 for a high-level summa
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-4351,IMPALA-4353: [qgen] randomly generate INSERT
statements
..
IMPALA-4351,IMPALA-4353: [qgen] randomly generate INSERT statements
- Ge
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-2615: warn if Status is ignored
..
Patch Set 7: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/4878
To unsubscribe, visit http://ge
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-2615: warn if Status is ignored
..
IMPALA-2615: warn if Status is ignored
This introduces a WARN_UNUSED_RESULT macro. It can be used at t
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-4752: make ObjectPool more efficient
..
IMPALA-4752: make ObjectPool more efficient
Previously it was implemented as a vector of pointers
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4752: make ObjectPool more efficient
..
Patch Set 4: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5666
To unsubscribe, visit http
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
IMPALA-4716: Expr rewrite causes IllegalStateException
The DECODE constructor in CaseE
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4716: Expr rewrite causes IllegalStateException
..
Patch Set 5: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5631
To unsubscribe,
Impala Public Jenkins has submitted this change and it was merged.
Change subject: IMPALA-4757: addendum: avoid double underscore in name
..
IMPALA-4757: addendum: avoid double underscore in name
Names containing double undersc
Impala Public Jenkins has posted comments on this change.
Change subject: IMPALA-4757: addendum: avoid double underscore in name
..
Patch Set 3: Verified+1
--
To view, visit http://gerrit.cloudera.org:8080/5702
To unsubscribe,
83 matches
Mail list logo