[kudu-CR] subprocess: add ScopedSubprocess

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: subprocess: add ScopedSubprocess .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6741/1/src/kudu/util/subprocess.h File

[kudu-CR] log block manager: corruptor test utility

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit

[kudu-CR] KUDU-1978: avoid corruption when deleting misaligned blocks

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1978: avoid corruption when deleting misaligned blocks .. Patch Set 3: (2 comments)

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to pad UNIXTIME_MICROS slots in scan results This changes the wire protocol to, upon

[kudu-CR] Silence clang -Waddress-of-packed-member warning in concurrent btree.h

2017-04-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Silence clang -Waddress-of-packed-member warning in concurrent_btree.h .. Patch Set 2: (1 comment)

[kudu-CR] Silence clang -Waddress-of-packed-member warning in concurrent btree.h

2017-04-20 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6693 to look at the new patch set (#2). Change subject: Silence clang -Waddress-of-packed-member warning in concurrent_btree.h

[kudu-CR](gh-pages) FAQ refresh

2017-04-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: FAQ refresh .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6697/1/faq.md File faq.md: PS1, Line 250: Kudu hasn't been officially tested

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-20 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#10). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-20 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 9: (2 comments)

[kudu-CR] WIP: Expose "raw" mode in KuduScanner and allow to pass flags

2017-04-19 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6624 to look at the new patch set (#5). Change subject: WIP: Expose "raw" mode in KuduScanner and allow to pass flags ..

[kudu-CR] block manager: gflag to control repairs at startup

2017-04-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: block manager: gflag to control repairs at startup .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6682/1//COMMIT_MSG Commit Message:

[kudu-CR] log block manager: corruptor test utility

2017-04-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit

[kudu-CR] fs: generate report during Open

2017-04-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6581 To unsubscribe, visit

[kudu-CR] Silence clang -Waddress-of-packed-member warning

2017-04-19 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6693 Change subject: Silence clang -Waddress-of-packed-member warning .. Silence clang -Waddress-of-packed-member warning macOS builds

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/8/src/kudu/common/wire_protocol.h

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#9). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Slice::ToDebugString() - Add option to not ascii'ize

2017-04-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6671 Change subject: Slice::ToDebugString() - Add option to not ascii'ize .. Slice::ToDebugString() - Add option to not ascii'ize This

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 8: (1 comment) http://gerrit.cloudera.org:8080/#/c/6623/7/src/kudu/common/wire_protocol.h

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-18 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#8). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#7). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (6 comments)

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 4: (3 comments)

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#6). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Patch Set 5: (2 comments) MJ: Still need to address some comments, but included the client-side fix

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-17 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#5). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] log block manager: corruptor test utility

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit

[kudu-CR] fs: generate report during Open

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 7: Code-Review+2 (2 comments)

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment)

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment)

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose a way to set "advanced" non-types scan options .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6624/4//COMMIT_MSG Commit

[kudu-CR] fs: generate report during Open

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/log_block_manager-test.cc File

[kudu-CR] log block manager: use extent maps to decide whether to truncate containers

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: use extent maps to decide whether to truncate containers .. Patch Set 5: Code-Review+2 -- To view, visit

[kudu-CR] env: add RWFile::GetExtentMap for analyzing file extents

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: env: add RWFile::GetExtentMap for analyzing file extents .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6583 To

[kudu-CR] log block manager: corruptor test utility

2017-04-13 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-13 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6624 to look at the new patch set (#4). Change subject: WIP: Expose a way to set "advanced" non-types scan options ..

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#4). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/13/src/kudu/client/scan_batch.h File

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has submitted this change and it was merged. Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get the raw data from a KuduScanBatch This allows to fetch both the direct and the

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 13: -Code-Review Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/6574 To

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 13: Code-Review+2 Unrelated failure of: RaftConsensusITest.TestKUDU_597 -- To view, visit

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6623 to look at the new patch set (#3). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6624 to look at the new patch set (#3). Change subject: WIP: Expose a way to set "advanced" non-types scan options ..

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: WIP: Expose a way to set "advanced" non-types scan options .. WIP: Expose a way to set "advanced" non-types scan options This adds a new method to

[kudu-CR] Allow to pad UNIXTIME MICROS slots in scan results

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: Allow to pad UNIXTIME_MICROS slots in scan results .. Allow to pad UNIXTIME_MICROS slots in scan results This changes the wire protocol to, upon request,

[kudu-CR] WIP: Expose a way to set "advanced" non-types scan options

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6624 Change subject: WIP: Expose a way to set "advanced" non-types scan options .. WIP: Expose a way to set "advanced" non-types scan

[kudu-CR] Allow to pad UNIXTIME MICROS slows in scan results

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6623 Change subject: Allow to pad UNIXTIME_MICROS slows in scan results .. Allow to pad UNIXTIME_MICROS slows in scan results This

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#13). Change subject: Allow to get the raw data from a KuduScanBatch

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-12 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/11/src/kudu/client/client-test.cc

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#12). Change subject: Allow to get the raw data from a KuduScanBatch

[kudu-CR] Allow to release an rpc transfer's data

2017-04-12 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6592 to look at the new patch set (#8). Change subject: Allow to release an rpc transfer's data .. Allow to release an

[kudu-CR] Allow to release an rpc transfer's data

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to release an rpc transfer's data .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/6592/6/src/kudu/rpc/outbound_call.h File

[kudu-CR] Allow to release an rpc transfer's data

2017-04-11 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6592 to look at the new patch set (#7). Change subject: Allow to release an rpc transfer's data .. Allow to release an

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 9: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/9/src/kudu/client/scan_batch.h File

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-11 Thread David Ribeiro Alves (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#10). Change subject: Allow to get the raw data from a KuduScanBatch

[kudu-CR] Allow to release an rpc transfer's data

2017-04-11 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6592 to look at the new patch set (#6). Change subject: Allow to release an rpc transfer's data .. Allow to release an

[kudu-CR] Allow to release an rpc transfer's data

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to release an rpc transfer's data .. Patch Set 5: (6 comments) http://gerrit.cloudera.org:8080/#/c/6592/5//COMMIT_MSG Commit Message: PS5, Line

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-11 Thread David Ribeiro Alves (Code Review)
Hello Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#9). Change subject: Allow to get the raw data from a KuduScanBatch ..

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 7: (3 comments) http://gerrit.cloudera.org:8080/#/c/6574/7/src/kudu/client/client-test.cc

[kudu-CR] fuzz-itest: fix a test failure with scan-at-snapshot

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fuzz-itest: fix a test failure with scan-at-snapshot .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6494 To unsubscribe,

[kudu-CR] fs: generate report during Open

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 4: (15 comments) http://gerrit.cloudera.org:8080/#/c/6581/4//COMMIT_MSG Commit Message: PS4, Line 9: This

[kudu-CR] env: add RWFile::GetExtentMap for analyzing file extents

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: env: add RWFile::GetExtentMap for analyzing file extents .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6583 To

[kudu-CR] log block manager: corruptor test utility

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: corruptor test utility .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6582 To unsubscribe, visit

[kudu-CR] log block manager: use extent maps to decide whether to truncate containers

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: use extent maps to decide whether to truncate containers .. Patch Set 4: Code-Review+2 -- To view, visit

[kudu-CR] log block manager: use extent maps to decide whether to truncate containers

2017-04-11 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: use extent maps to decide whether to truncate containers .. Patch Set 3: (1 comment)

[kudu-CR] log block manager: use extent maps to decide whether to truncate containers

2017-04-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: use extent maps to decide whether to truncate containers .. Patch Set 3: (3 comments)

[kudu-CR] env: add RWFile::GetExtentMap for analyzing file extents

2017-04-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: env: add RWFile::GetExtentMap for analyzing file extents .. Patch Set 3: (4 comments)

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 6: (4 comments) http://gerrit.cloudera.org:8080/#/c/6574/6/src/kudu/client/client-test.cc

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#7). Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#6). Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/4/src/kudu/client/scan_batch.h File

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#5). Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/3/src/kudu/client/scan_batch.h File

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6574 to look at the new patch set (#4). Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get

[kudu-CR] Allow to release an rpc transfer's data

2017-04-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6592 to look at the new patch set (#5). Change subject: Allow to release an rpc transfer's data .. Allow to release an

[kudu-CR] fs: ensure FS IOC FIEMAP can be used on LBM systems

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: ensure FS_IOC_FIEMAP can be used on LBM systems .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6584/1/src/kudu/fs/data_dirs.cc File

[kudu-CR] fs: ensure FS IOC FIEMAP can be used on LBM systems

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: ensure FS_IOC_FIEMAP can be used on LBM systems .. Patch Set 1: Code-Review+2 (1 comment) feel free to ignore the nit or address it

[kudu-CR] env: add RWFile::GetExtentMap for analyzing file extents

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: env: add RWFile::GetExtentMap for analyzing file extents .. Patch Set 1: (10 comments)

[kudu-CR] fs: generate report during Open

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: fs: generate report during Open .. Patch Set 1: (11 comments) first round of reviews. this patch is pretty heftly so I'll review it in 2-3 rounds or

[kudu-CR] [consensus] fixed typos in consensus flags tagging

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [consensus] fixed typos in consensus flags tagging .. Patch Set 1: (1 comment)

[kudu-CR] WIP: Allow to release an rpc transfer's data

2017-04-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6592 to look at the new patch set (#4). Change subject: WIP: Allow to release an rpc transfer's data .. WIP: Allow to

[kudu-CR] WIP: Allow to release an rpc transfer's data

2017-04-07 Thread David Ribeiro Alves (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6592 to look at the new patch set (#3). Change subject: WIP: Allow to release an rpc transfer's data .. WIP: Allow to

[kudu-CR] WIP: Allow to release an rpc transfer's data

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: WIP: Allow to release an rpc transfer's data .. WIP: Allow to release an rpc transfer's data This adds a way to release a transfer's data to a caller. This

[kudu-CR] whole batch release, but wrong

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has abandoned this change. Change subject: whole batch release, but wrong .. Abandoned oops forgot to squash -- To view, visit http://gerrit.cloudera.org:8080/6591 To unsubscribe, visit

[kudu-CR] whole batch release, but wrong

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6591 Change subject: whole batch release, but wrong .. whole batch release, but wrong Change-Id:

[kudu-CR] WIP: Allow to release an rpc transfer's data

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6592 Change subject: WIP: Allow to release an rpc transfer's data .. WIP: Allow to release an rpc transfer's data This adds a way to

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/1/src/kudu/client/scan_batch.h File

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#3). Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get the raw data from a KuduScanBatch This allows to fetch both the direct and the indirect raw

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new patch set (#2). Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get the raw data from a KuduScanBatch This allows to fetch both the direct and the indirect raw

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-07 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 1: ok, makes sense. will do that in a follow up patch. -- To view, visit

[kudu-CR] linked list-test: dump a histogram of Update performance

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: linked_list-test: dump a histogram of Update performance .. Patch Set 1: (2 comments)

[kudu-CR] linked list-test: dump a histogram of Update performance

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: linked_list-test: dump a histogram of Update performance .. Patch Set 1: (1 comment)

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 1: Verified+1 Unrelated flake. -- To view, visit http://gerrit.cloudera.org:8080/6574 To

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Allow to get the raw data from a KuduScanBatch .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/1/src/kudu/client/scan_batch.h File

[kudu-CR] Allow to get the raw data from a KuduScanBatch

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has uploaded a new change for review. http://gerrit.cloudera.org:8080/6574 Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get the raw data from a KuduScanBatch This allows to

[kudu-CR] [kudu-jepsen] install Kudu packages into local repo

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [kudu-jepsen] install Kudu packages into local repo .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6256 To unsubscribe,

[kudu-CR] [kudu-jepsen] install Kudu packages into local repo

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [kudu-jepsen] install Kudu packages into local repo .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6256/2/src/kudu/scripts/jepsen.sh

[kudu-CR] [kudu-jepsen] install Kudu packages into local repo

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [kudu-jepsen] install Kudu packages into local repo .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6256/1/src/kudu/scripts/jepsen.sh

[kudu-CR] [kudu-jepsen] updated client model for Jepsen tests

2017-04-06 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [kudu-jepsen] updated client model for Jepsen tests .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6565 To unsubscribe,

[kudu-CR] tablet: check for row presence in rowset-wise order

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: tablet: check for row presence in rowset-wise order .. Patch Set 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6483 To unsubscribe,

<    2   3   4   5   6   7   8   9   10   11   >