[kudu-CR] subprocess: add KillAndWait() and allow customization of exit signal

2017-04-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: subprocess: add KillAndWait() and allow customization of exit signal .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6741

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

2017-04-28 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 11: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6583 To unsubscr

[kudu-CR] log block manager: detect and repair unpunched holes

2017-04-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: detect and repair unpunched holes .. Patch Set 5: (11 comments) http://gerrit.cloudera.org:8080/#/c/6717/5/src/kudu/fs/log_block_manage

[kudu-CR] log block manager: detect and repair unpunched holes

2017-04-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: detect and repair unpunched holes .. Patch Set 5: Adding Andrew to the review since this touches stuff he might also be touching -- To

[kudu-CR] log block manager: more container accounting

2017-04-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: log block manager: more container accounting .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6716 To unsubscribe, visit ht

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

2017-04-28 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 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6715 To unsubs

[kudu-CR] external mini cluster: spawn perf record for each daemon during Start()

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: external mini cluster: spawn perf record for each daemon during Start() .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6742/1/src/kudu/

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

2017-04-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: FAQ refresh .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6697 To unsubscribe, visit http://gerrit.cloudera.org:8080/set

[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 src/kudu/util/subp

[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 http

[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) http://gerrit.cloudera.org:8080/#/c/6715/3/src/kudu/fs/log_block_

[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 r

[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) http://gerrit.cloudera.org:8080/#/c/6693/2/src/kudu/ta

[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) http://gerrit.cloudera.org:8080/#/c/6623/9/src/kudu/common/wire_protocol-t

[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: PS

[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 http:

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

2017-04-19 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 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6583 To unsubscri

[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 http://gerrit.c

[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 p

[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 p

[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 p

[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) http://gerrit.cloudera.org:8080/#/c/6623/4/src/kudu/common/wire_protocol-t

[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) http://gerrit.cloudera.org:8080/#/c/6623/4/src/kudu/common/wire_protocol.c

[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 p

[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 p

[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 http:

[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 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6583 To unsubscri

[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) http://gerrit.cloudera.org:8080/#/c/6581/4/src/kudu/fs/log_block_manager-test.c

[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/src/kudu/tserver/tserver

[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/src/kudu/client/scan_con

[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 Messa

[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 Messa

[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 src/kud

[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 http://gerrit.cloudera.org:80

[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 unsubscri

[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 http:

[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 (#4). Change subject: WIP: Expose a way to set "advanced" non-types scan options .. WI

[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 p

[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 indi

[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 unsubscr

[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 htt

[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 p

[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 .. WI

[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 KuduScanne

[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, pad

[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 op

[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 chan

[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 Fil

[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 r

[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 src/kudu/

[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 r

[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 sr

[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 r

[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 10:

[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 File

[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 pa

[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 unsubscri

[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 http:

[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 http://gerrit.cloudera.org:80

[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) http://gerrit.cloudera.org:8080/#/c/6585/3/src/ku

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

2017-04-10 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 3: (17 comments) http://gerrit.cloudera.org:8080/#/c/6582/3/src/kudu/fs/block_manager-stress-test.c

[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) http://gerrit.cloudera.org:8080/#/c/6585/3/src/k

[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) http://gerrit.cloudera.org:8080/#/c/6583/1/src/kudu/util/env-test.cc

[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 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 (#7). Change subject: Allow to get the raw data from a KuduScanBatch .. Allow to get t

[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 t

[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 sr

[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 t

[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 sr

[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 t

[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 r

[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 s

[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 http://gerrit.clo

[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) http://gerrit.cloudera.org:8080/#/c/6583/1/src/kudu/util/env-test.c

[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 less

[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) http://gerrit.cloudera.org:8080/#/c/6580/1/src/kudu/consensus/consensus_pee

[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 re

[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 re

[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 i

[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 http://gerrit.cloud

[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: Ib2080a9832b096b13a1a5b9ce88ef522e30b0

[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 re

[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 sr

[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 da

[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 da

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