David Ribeiro Alves has posted comments on this change.
Change subject: env: Add support for getting FS capacity
..
Patch Set 2: Code-Review+2
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6255/1/src/kudu/util/env-test.cc
David Ribeiro Alves has posted comments on this change.
Change subject: env: Add support for getting FS capacity
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6255/1/src/kudu/util/env-test.cc
File
David Ribeiro Alves has posted comments on this change.
Change subject: [catalog_manager] categorization of rw operation failures
..
Patch Set 10:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/6170/10//COMMIT_MSG
Commit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6258
to look at the new patch set (#4).
Change subject: KUDU-1905 - Allow reinserts on pk only tables
..
KUDU-1905 -
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1905 - Allow reinserts on pk only tables
..
Patch Set 2:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1905 - Allow reinserts on pk only tables
..
Patch Set 3:
(5 comments)
http://gerrit.cloudera.org:8080/#/c/6258/1//COMMIT_MSG
Commit Message:
PS1,
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6258
to look at the new patch set (#3).
Change subject: KUDU-1905 - Allow reinserts on pk only tables
..
KUDU-1905 -
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6258
to look at the new patch set (#2).
Change subject: KUDU-1905 - Allow reinserts on pk only tables
..
KUDU-1905 -
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6258
Change subject: WIP: KUDU-1905 - Allow reinserts on pk only tables
..
WIP: KUDU-1905 - Allow reinserts on pk only tables
Doing a
David Ribeiro Alves has submitted this change and it was merged.
Change subject: KUDU-1893 Ensure evaluation of added columns
..
KUDU-1893 Ensure evaluation of added columns
During a normal scan, a CFileIterator sets a flag to
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1893 Ensure evaluation of added columns
..
Patch Set 1: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6225
To unsubscribe, visit
David Ribeiro Alves has submitted this change and it was merged.
Change subject: [flaky tests] Allow for the master to take longer to start
..
[flaky tests] Allow for the master to take longer to start
With the additional work
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6211
to look at the new patch set (#6).
Change subject: Make ExternalDaemon::StartProcess() handle fault injection
..
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6230
Change subject: [flaky tests] Allow for the master to take longer to start
..
[flaky tests] Allow for the master to take longer to
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6211
to look at the new patch set (#4).
Change subject: Make ExternalDaemon::StartProcess() handle fault injection
..
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6211
to look at the new patch set (#5).
Change subject: Make ExternalDaemon::StartProcess() handle fault injection
..
David Ribeiro Alves has uploaded a new patch set (#2).
Change subject: [ts_recovery-itest] Reduce the number of rows written in
TestChangeMaxCellSize
..
[ts_recovery-itest] Reduce the number of rows written in
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6227
Change subject: [ts_recovery-itest] Reduce the number of rows written
..
[ts_recovery-itest] Reduce the number of rows written
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6211
to look at the new patch set (#3).
Change subject: Make ExternalDaemon::StartProcess() handle fault injection
..
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6211
to look at the new patch set (#2).
Change subject: Make ExternalDaemon::StartProcess() handle fault injection
..
David Ribeiro Alves has submitted this change and it was merged.
Change subject: KUDU-1893 Ensure evaluation of added columns
..
KUDU-1893 Ensure evaluation of added columns
During a normal scan, a CFileIterator sets a flag to
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1893 Ensure evaluation of added columns
..
Patch Set 8:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/6129/8//COMMIT_MSG
Commit Message:
PS8,
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6211
Change subject: WIP: Make ExternalDaemon::StartProcess() handle fault injection
..
WIP: Make ExternalDaemon::StartProcess() handle
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1893 Ensure evaluation of added columns
..
Patch Set 6:
(9 comments)
http://gerrit.cloudera.org:8080/#/c/6129/6/src/kudu/common/column_predicate.cc
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6200
Change subject: [delete_table-test] Disable log compression on some tests
..
[delete_table-test] Disable log compression on some
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6198
Change subject: [client-test] Reduce flakyness of TestWriteWithDeadTabletServer
..
[client-test] Reduce flakyness of
David Ribeiro Alves has uploaded a new patch set (#2).
Change subject: [delete_table-test] Don't fail on multiple attempts to delete
an orphaned tablet
..
[delete_table-test] Don't fail on multiple attempts to delete an
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/6191
Change subject: [delete_table-test] Don't fail on multiple attempts to delete
an orphaned tablet
..
[delete_table-test] Don't
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1880 Prevent eager ignoring of NULLs
..
Patch Set 11: Code-Review+2
lgtm, assuming tests pass
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1880 Prevent eager ignoring of NULLs
..
Patch Set 9: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6029
To unsubscribe, visit
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1904 Don't seek on empty RLE blocks
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6184/4/src/kudu/cfile/rle_block.h
File
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1904 Don't seek on empty RLE blocks
..
Patch Set 3:
lgtm, just waiting for the commit that has coverage for this to +2
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1904 Don't seek on empty RLE blocks
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6184/2/src/kudu/cfile/rle_block.h
File
David Ribeiro Alves has posted comments on this change.
Change subject: Misc optimizations to BinaryPlainBlockDecoder
..
Patch Set 4: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6159
To unsubscribe, visit
David Ribeiro Alves has posted comments on this change.
Change subject: Misc optimizations to BinaryPlainBlockDecoder
..
Patch Set 4:
doh! yeah, wrong rev. sorry about that
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: Misc optimizations to BinaryPlainBlockDecoder
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6159/2/src/kudu/cfile/binary_plain_block.cc
David Ribeiro Alves has posted comments on this change.
Change subject: Micro-optimizations to try to inline DeltaKey::DecodeFrom
..
Patch Set 3: Code-Review+2
(1 comment)
tiny nit, feel free to +2 when you fix (or if my
David Ribeiro Alves has posted comments on this change.
Change subject: [catalog_manager] proper handling of catalog shutdown
..
Patch Set 4:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1899. Fix support for empty string keys
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6163
To unsubscribe, visit
David Ribeiro Alves has posted comments on this change.
Change subject: WIP KUDU-1330: Add a tool to unsafely recover from loss of
majority replicas
..
Patch Set 1:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: [catalog manager] fixed deadlock on catalog shutdown
..
Patch Set 8: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6134
To unsubscribe,
David Ribeiro Alves has posted comments on this change.
Change subject: Micro-optimizations to try to inline DeltaKey::DecodeFrom
..
Patch Set 1:
(2 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: Misc optimizations to BinaryPlainBlockDecoder
..
Patch Set 2:
(2 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: [catalog manager] fixed deadlock on catalog shutdown
..
Patch Set 4:
(16 comments)
http://gerrit.cloudera.org:8080/#/c/6134/4//COMMIT_MSG
Commit Message:
David Ribeiro Alves has posted comments on this change.
Change subject: WIP: [catalog manager] fixed deadlock on catalog shutdown
..
Patch Set 2:
(7 comments)
http://gerrit.cloudera.org:8080/#/c/6134/2//COMMIT_MSG
Commit
David Ribeiro Alves has posted comments on this change.
Change subject: WIP: [catalog manager] fixed deadlock on catalog shutdown
..
Patch Set 2:
could you post a failure trace in the commit message?
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: [catalog manager] fixed deadlock on catalog shutdown
..
Patch Set 1:
do you have a test that reproduces this bug? not saying we absolutely need it
but it
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 20: Code-Review+2
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1880 Prevent eager ignoring of NULLs
..
Patch Set 4:
(2 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1880 Prevent eager ignoring of NULLs
..
Patch Set 4:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/6029/4/src/kudu/cfile/binary_dict_block.h
File
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1893 Ensure evaluation of added columns
..
Patch Set 2:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/6129/2/src/kudu/cfile/cfile_reader.cc
File
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 19: Code-Review+1
code lgtm, just waiting on those dist-test/cluster test
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1893 Ensure evaluation of added columns
..
Patch Set 2:
(13 comments)
http://gerrit.cloudera.org:8080/#/c/6129/2/src/kudu/cfile/cfile_reader.cc
File
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 18:
(13 comments)
http://gerrit.cloudera.org:8080/#/c/4363/18//COMMIT_MSG
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 17:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: master: Do not acquire lock_ during CatalogManager::Init()
..
Patch Set 4:
Which follow up patch? Was this related to the "server-wide" op? Now that we
David Ribeiro Alves has posted comments on this change.
Change subject: client-test-util: Increase ScanTableToStrings() timeout
..
Patch Set 2: Code-Review+2
--
To view, visit http://gerrit.cloudera.org:8080/6104
To
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 16:
(7 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 13:
(11 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 12:
(14 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 11:
(2 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 9:
(9 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1601. Delete ancient UNDO delta blocks in the background
..
Patch Set 9:
(23 comments)
http://gerrit.cloudera.org:8080/#/c/4363/9//COMMIT_MSG
Commit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5965
to look at the new patch set (#8).
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 6:
(1 comment)
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5965
to look at the new patch set (#7).
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
David Ribeiro Alves has uploaded a new patch set (#7).
Change subject: [timestamp] Add a new TIMESTAMP_NANOS type
..
[timestamp] Add a new TIMESTAMP_NANOS type
This adds a new type to Kudu and adds the ability to use
it in
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 6:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 6:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 6:
(1 comment)
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 5:
(7 comments)
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5965
to look at the new patch set (#6).
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TIMESTAMP_NANOS type
..
Patch Set 6: Verified+1
unrelated flake RpcBench.BenchmarkCallsAsync
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: Add a helper method to determine whether the OS is macOS to
KuduTest
..
Patch Set 3: Verified+1
unrelated flake
--
To view, visit
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5819
to look at the new patch set (#6).
Change subject: [timestamp] Add a new TIMESTAMP_NANOS type
..
[timestamp] Add a
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5965
to look at the new patch set (#5).
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 5: Verified+1
unrelated flake
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 4:
(3 comments)
Hello Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5981
to look at the new patch set (#3).
Change subject: Add a helper method to determine whether the OS is macOS to
KuduTest
David Ribeiro Alves has posted comments on this change.
Change subject: Add a helper method to determine whether the OS is macOS to
KuduTest
..
Patch Set 1:
(1 comment)
Hello Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5981
to look at the new patch set (#2).
Change subject: Add a helper method to determine whether the OS is macOS to
KuduTest
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
Patch Set 3:
(16 comments)
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TIMESTAMP_NANOS type
..
Patch Set 5:
(3 comments)
http://gerrit.cloudera.org:8080/#/c/5819/3/src/kudu/client/schema.h
File
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5819
to look at the new patch set (#5).
Change subject: [timestamp] Add a new TIMESTAMP_NANOS type
..
[timestamp] Add a
David Ribeiro Alves has uploaded a new patch set (#4).
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP_NANOS type
..
[timestamp] Add a new TimestampValue class to support the TIMESTAMP_NANOS
David Ribeiro Alves has uploaded a new patch set (#3).
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP type
..
[timestamp] Add a new TimestampValue class to support the TIMESTAMP type
This
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1870: Fix Boost build to respect CC/CXX
..
Patch Set 4:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5986/4/thirdparty/build-definitions.sh
File
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1870: Fix Boost build to respect CC/CXX
..
Patch Set 2:
(2 comments)
http://gerrit.cloudera.org:8080/#/c/5986/2/thirdparty/build-definitions.sh
File
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP type
..
Patch Set 2: Verified+1
unrelated java flake
--
To view, visit
David Ribeiro Alves has posted comments on this change.
Change subject: Refactor HybridClock and add a local clock mode
..
Patch Set 1:
(1 comment)
todd: just for internal tests and hidden + unsafe, yeah.
mj: my problem with
David Ribeiro Alves has posted comments on this change.
Change subject: KUDU-1870: Fix Boost build to respect CC/CXX
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5986/1/thirdparty/build-definitions.sh
File
David Ribeiro Alves has posted comments on this change.
Change subject: Refactor HybridClock and add a local clock mode
..
Patch Set 1:
(1 comment)
http://gerrit.cloudera.org:8080/#/c/5982/1/src/kudu/server/hybrid_clock.cc
David Ribeiro Alves has posted comments on this change.
Change subject: [timestamp] Add a new TIMESTAMP type
..
Patch Set 4:
(7 comments)
http://gerrit.cloudera.org:8080/#/c/5819/3/src/kudu/client/schema.h
File
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5819
to look at the new patch set (#4).
Change subject: [timestamp] Add a new TIMESTAMP type
..
[timestamp] Add a new
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5982
Change subject: Refactor HybridClock and add a local clock mode
..
Refactor HybridClock and add a local clock mode
Some test
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5981
Change subject: Add a helper method to determine whether the OS is macOS to
KuduTest
..
Add a helper method to determine whether
David Ribeiro Alves has posted comments on this change.
Change subject: WIP: Add a new TIMESTAMP type
..
Patch Set 3:
(44 comments)
Addressed all comments on the TimestampValue class (posted in another patch).
Remaining
David Ribeiro Alves has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5965
Change subject: [timestamp] Add a new TimestampValue class to support the
TIMESTAMP type
..
[timestamp] Add a new TimestampValue
David Ribeiro Alves has posted comments on this change.
Change subject: Add Google Breakpad support to Kudu
..
Patch Set 37: Code-Review+2
haven't actually tested this last rev on mac, assuming there weren't any new
breakages
Hello Mike Percy, Adar Dembo, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5620
to look at the new patch set (#34).
Change subject: Add Google Breakpad support to Kudu
..
801 - 900 of 1602 matches
Mail list logo