[kudu-CR] write transaction: allocate RowOp objects from the arena

2017-03-27 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: write_transaction: allocate RowOp objects from the arena .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6480 To unsubscri

[kudu-CR] interval tree: improve an O(n) loop to O(lg n)

2017-03-28 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: interval_tree: improve an O(n) loop to O(lg n) .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6496/1/src/kudu/util/interval_tree-inl.h Fi

[kudu-CR] interval tree: improve an O(n) loop to O(lg n)

2017-03-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: interval_tree: improve an O(n) loop to O(lg n) .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6496 To unsubscribe, visit

[kudu-CR] interval tree: allow bulk queries

2017-03-29 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: interval_tree: allow bulk queries .. Patch Set 4: (5 comments) http://gerrit.cloudera.org:8080/#/c/6481/4/src/kudu/util/interval_tree-inl.h File src/kudu/

[kudu-CR] rowset tree: add bulk queries

2017-03-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: rowset_tree: add bulk queries .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/6482/4/src/kudu/tablet/rowset_tree-test.cc File src/kudu/ta

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

2017-03-30 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 8: (14 comments) http://gerrit.cloudera.org:8080/#/c/6483/8/src/kudu/tablet/row_op.h File

[kudu-CR] cfile-test: add a 100M-string file with low cardinality

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: cfile-test: add a 100M-string file with low cardinality .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6432 To unsubscrib

[kudu-CR] generic iterators: avoid holding completed iters in merge/union

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: generic_iterators: avoid holding completed iters in merge/union .. Patch Set 3: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6460/1//COMM

[kudu-CR] interval tree: allow bulk queries

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: interval_tree: allow bulk queries .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/6481/4/src/kudu/util/interval_tree-test.cc File src/kud

[kudu-CR] rowset tree: add bulk queries

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: rowset_tree: add bulk queries .. Patch Set 6: Code-Review+2 (1 comment) yeah, it seems that this always outperforms the 1-by-1 version for the cases that

[kudu-CR] interval tree: allow bulk queries

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: interval_tree: allow bulk queries .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6481/4/src/kudu/util/interval_tree.h File src/kudu/util/

[kudu-CR] interval tree: allow bulk queries

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: interval_tree: allow bulk queries .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6481 To unsubscribe, visit http://gerrit

[kudu-CR] generic iterators: avoid holding completed iters in merge/union

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: generic_iterators: avoid holding completed iters in merge/union .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6460 To un

[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 10: (8 comments) http://gerrit.cloudera.org:8080/#/c/6483/10//COMMIT_MSG Commit Message:

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

2017-04-05 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 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6565/1//COMMIT_MSG Commit Message: P

[kudu-CR] [kudu-jepsen] clean up on require/import lists

2017-04-05 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [kudu-jepsen] clean up on require/import lists .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6570 To unsubscribe, visit

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

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

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

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

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

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

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

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

[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) http://gerrit.cloudera.org:8080/#/c/6576/1/src/kudu/integration-tests

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

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

[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

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

[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] 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 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 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] 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] 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] 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 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: (1 comment) http://gerrit.cloudera.org:8080/#/c/6574/13/src/kudu/client/scan_batch.h File

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

[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] 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] 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] 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 (#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 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)
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)
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-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

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