[kudu-CR] tablet copy: Ensure no data loss on tablet copy failure

Alexey Serbin has posted comments on this change. Change subject: tablet copy: Ensure no data loss on tablet copy failure .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6732 To unsubscribe, vis

[kudu-CR] Expose row format flags in KuduScanner

Alexey Serbin has posted comments on this change. Change subject: Expose row format flags in KuduScanner .. Patch Set 8: (8 comments) Just skimmed over the code to get more context on the new feature. http://gerrit.cloudera.o

[kudu-CR] spark: add support for fault tolerant scanner

Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6782 Change subject: spark: add support for fault tolerant scanner .. spark: add support for fault tolerant scanner This adds support to use fault

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

Grant Henke has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 13: (25 comments) http://gerrit.cloudera.org:8080/#/c/6630/13/src/kudu/cfile/cfile-test.cc File src/kudu/cfi

[kudu-CR] env: add ReadV() API

Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6779/2/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: Line 330:

[kudu-CR] Expose row format flags in KuduScanner

David Ribeiro Alves has posted comments on this change. Change subject: Expose row format flags in KuduScanner .. Patch Set 8: Verified+1 unrelated ITClient test failure -- To view, visit http://gerrit.cloudera.org:8080/6624

[kudu-CR] Expose row format flags in KuduScanner

David Ribeiro Alves has uploaded a new patch set (#8). Change subject: Expose row format flags in KuduScanner .. Expose row format flags in KuduScanner This adds a way to pass row format modifier flags to KuduScanner, encoded in

[kudu-CR] Expose row format flags in KuduScanner

David Ribeiro Alves has posted comments on this change. Change subject: Expose row format flags in KuduScanner .. Patch Set 7: (5 comments) http://gerrit.cloudera.org:8080/#/c/6624/7/src/kudu/integration-tests/all_types-itest.

[kudu-CR] [java client] improve flaky test ITClient

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6776 to look at the new patch set (#3). Change subject: [java client] improve flaky test ITClient .. [java client] impro

[kudu-CR] [java client] improve flaky test ITClient

Hao Hao has posted comments on this change. Change subject: [java client] improve flaky test ITClient .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/6776/2/java/kudu-client/src/main/java/org/apache/kudu/clien

[kudu-CR] Expose row format flags in KuduScanner

David Ribeiro Alves has posted comments on this change. Change subject: Expose row format flags in KuduScanner .. Patch Set 7: Verified+1 unrelated failure -- To view, visit http://gerrit.cloudera.org:8080/6624 To unsubscribe

[kudu-CR] Expose row format flags in KuduScanner

David Ribeiro Alves has uploaded a new patch set (#7). Change subject: Expose row format flags in KuduScanner .. Expose row format flags in KuduScanner This adds a way to pass row format modifier flags to KuduScanner, encoded in

[kudu-CR] Expose row format flags in KuduScanner

David Ribeiro Alves has posted comments on this change. Change subject: Expose row format flags in KuduScanner .. Patch Set 6: (9 comments) http://gerrit.cloudera.org:8080/#/c/6624/6/src/kudu/client/client.cc File src/kudu/cli

[kudu-CR] [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config

Todd Lipcon has posted comments on this change. Change subject: [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config .. Patch Set 4: Just passing through, haven't looked at the code yet, but one

[kudu-CR] env: add ReadV() API

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6779 to look at the new patch set (#2). Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() metho

[kudu-CR] [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config

Mike Percy has posted comments on this change. Change subject: [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config .. Patch Set 4: (4 comments) http://gerrit.cloudera.org:8080/#/c/6772/4/src/ku

[kudu-CR] design docs: Add tablet history GC design doc

Mike Percy has posted comments on this change. Change subject: design docs: Add tablet history GC design doc .. Patch Set 6: Verified+1 Overriding build failure due to ITClient hang -- To view, visit http://gerrit.cloudera.or

[kudu-CR] tablet copy: Ensure no data loss on tablet copy failure

Adar Dembo has posted comments on this change. Change subject: tablet copy: Ensure no data loss on tablet copy failure .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6732 To unsubscribe, visit

[kudu-CR] tablet copy: Ensure no data loss on tablet copy failure

Mike Percy has posted comments on this change. Change subject: tablet copy: Ensure no data loss on tablet copy failure .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6732/5/src/kudu/integration-tests/tablet_co

[kudu-CR] tablet copy: Ensure no data loss on tablet copy failure

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6732 to look at the new patch set (#6). Change subject: tablet copy: Ensure no data loss on tablet copy failure .. table

[kudu-CR] design docs: Add tablet history GC design doc

Mike Percy has posted comments on this change. Change subject: design docs: Add tablet history GC design doc .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/2853/5/docs/design-docs/tablet-history-gc.md File doc

[kudu-CR] design docs: Add tablet history GC design doc

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/2853 to look at the new patch set (#6). Change subject: design docs: Add tablet history GC design doc .. design docs: Ad

[kudu-CR] Reduce arguments in file Read API

Adar Dembo has submitted this change and it was merged. Change subject: Reduce arguments in file Read API .. Reduce arguments in file Read API Because reads are now always read fully or result in an error if all bytes can’t be

[kudu-CR] [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config

Mike Percy has posted comments on this change. Change subject: [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/6772/1/src/ku

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

Adar Dembo has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 13: (1 comment) http://gerrit.cloudera.org:8080/#/c/6630/13/src/kudu/cfile/cfile_reader.cc File src/kudu/cfil

[kudu-CR] [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config

Mike Percy has posted comments on this change. Change subject: [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config .. Patch Set 4: > I implemented a GetAllConsensusStates RPC and used it instead

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

Adar Dembo has posted comments on this change. Change subject: WIP: KUDU-463. Add checksumming to cfile .. Patch Set 13: (28 comments) http://gerrit.cloudera.org:8080/#/c/6630/13/src/kudu/cfile/cfile-test.cc File src/kudu/cfil

[kudu-CR] tablet copy: Ensure no data loss on tablet copy failure

Adar Dembo has posted comments on this change. Change subject: tablet copy: Ensure no data loss on tablet copy failure .. Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/6732/5/src/kudu/integration-tests/tablet_co

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

Adar Dembo has submitted this change and it was merged. Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .. [KUDU-754] add an environment variable for kudu client debugging to stderr R

[kudu-CR] log block manager: refactor container record processing

Adar Dembo has submitted this change and it was merged. Change subject: log_block_manager: refactor container record processing .. log_block_manager: refactor container record processing While working on some other LBM stuff I

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

Adar Dembo has submitted this change and it was merged. Change subject: log block manager: detect and repair unpunched holes .. log block manager: detect and repair unpunched holes This patch adds detection and repair of "unpun

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

Alexey Serbin has posted comments on this change. Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6736

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

Adar Dembo has posted comments on this change. Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6736 To

[kudu-CR] tablet copy: Ensure no data loss on tablet copy failure

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6732 to look at the new patch set (#5). Change subject: tablet copy: Ensure no data loss on tablet copy failure .. table

[kudu-CR] tablet copy: Ensure no data loss on tablet copy failure

Mike Percy has posted comments on this change. Change subject: tablet copy: Ensure no data loss on tablet copy failure .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6732/4/src/kudu/integration-tests/tablet_co

[kudu-CR] tablet copy: Refactor to avoid separate expiration map

Mike Percy has submitted this change and it was merged. Change subject: tablet copy: Refactor to avoid separate expiration map .. tablet copy: Refactor to avoid separate expiration map This simplifies the implementation a bit b

[kudu-CR] tablet copy: Refactor to avoid separate expiration map

Mike Percy has posted comments on this change. Change subject: tablet copy: Refactor to avoid separate expiration map .. Patch Set 4: Code-Review+2 Porting over Adar's previous +2 after a rebase -- To view, visit http://gerri

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

William Li has posted comments on this change. Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .. Patch Set 5: (4 comments) http://gerrit.cloudera.org:8080/#/c/6736/4//COMMIT_MSG Com

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6736 to look at the new patch set (#5). Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .

[kudu-CR] env: add ReadV() API

Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: Line 312:

[kudu-CR] env: add ReadV() API

Grant Henke has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/fs/block_manager.h File src/kudu/fs/block_manager.h: Line 1

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

David Ribeiro Alves has posted comments on this change. Change subject: log block manager: detect and repair unpunched holes .. Patch Set 8: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6717 To unsubscribe,

[kudu-CR] Reduce arguments in file Read API

Adar Dembo has posted comments on this change. Change subject: Reduce arguments in file Read API .. Patch Set 7: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6770 To unsubscribe, visit http://gerrit.cloudera

[kudu-CR] Reduce arguments in file Read API

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6770 to look at the new patch set (#7). Change subject: Reduce arguments in file Read API .. Reduce arguments in file Re

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Alexey Serbin has posted comments on this change. Change subject: [webui] Fix master reg formatting and add memory use percentage. .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6780 To unsubsc

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

Adar Dembo has posted comments on this change. Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .. Patch Set 4: (4 comments) Looks good, thanks for the contribution! Just passing thro

[kudu-CR] Reduce arguments in file Read API

Grant Henke has posted comments on this change. Change subject: Reduce arguments in file Read API .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/6770/6/src/kudu/util/pb_util.cc File src/kudu/util/pb_util.cc:

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Will Berkeley has posted comments on this change. Change subject: [webui] Fix master reg formatting and add memory use percentage. .. Patch Set 4: Verified+1 Overriding Jenkins- unrelated failure. -- To view, visit http://ger

[kudu-CR] Reduce arguments in file Read API

Adar Dembo has posted comments on this change. Change subject: Reduce arguments in file Read API .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/6770/6/src/kudu/util/pb_util.cc File src/kudu/util/pb_util.cc: L

[kudu-CR] WIP: Expose row format flags in KuduScanner

Adar Dembo has posted comments on this change. Change subject: WIP: Expose row format flags in KuduScanner .. Patch Set 6: (9 comments) http://gerrit.cloudera.org:8080/#/c/6624/6/src/kudu/client/client.cc File src/kudu/client/

[kudu-CR] Reduce arguments in file Read API

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6770 to look at the new patch set (#6). Change subject: Reduce arguments in file Read API .. Reduce arguments in file Re

[kudu-CR] Reduce arguments in file Read API

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6770 to look at the new patch set (#5). Change subject: Reduce arguments in file Read API .. Reduce arguments in file Re

[kudu-CR] Reduce arguments in file Read API

Grant Henke has posted comments on this change. Change subject: Reduce arguments in file Read API .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6770/3/src/kudu/util/pb_util.cc File src/kudu/util/pb_util.cc:

[kudu-CR] env: add ReadV() API

Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/util/env_posix.cc File src/kudu/util/env_posix.cc: Line 312:

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Adar Dembo has posted comments on this change. Change subject: [webui] Fix master reg formatting and add memory use percentage. .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6780 To unsubscrib

[kudu-CR] env: add ReadV() API

Adar Dembo has posted comments on this change. Change subject: env: add ReadV() API .. Patch Set 1: (5 comments) http://gerrit.cloudera.org:8080/#/c/6779/1/src/kudu/fs/block_manager.h File src/kudu/fs/block_manager.h: PS1, Li

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Will Berkeley has posted comments on this change. Change subject: [webui] Fix master reg formatting and add memory use percentage. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6780/3/src/kudu/server/default-

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6780 to look at the new patch set (#4). Change subject: [webui] Fix master reg formatting and add memory use percentage.

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Alexey Serbin has posted comments on this change. Change subject: [webui] Fix master reg formatting and add memory use percentage. .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6780/3/src/kudu/server/default-

[kudu-CR] Reduce arguments in file Read API

Adar Dembo has posted comments on this change. Change subject: Reduce arguments in file Read API .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/6770/3/src/kudu/util/pb_util.cc File src/kudu/util/pb_util.cc: L

[kudu-CR] KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs

Alexey Serbin has posted comments on this change. Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs .. Patch Set 11: (3 comments) http://gerrit.cloudera.org:8080/#/c/6514/11/src/kud

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6780 to look at the new patch set (#3). Change subject: [webui] Fix master reg formatting and add memory use percentage.

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

William Li has posted comments on this change. Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .. Patch Set 4: (9 comments) http://gerrit.cloudera.org:8080/#/c/6736/2/src/kudu/client

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6736 to look at the new patch set (#4). Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .

[kudu-CR] WIP: KUDU-463. Add checksumming to cfile

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6630 to look at the new patch set (#13). Change subject: WIP: KUDU-463. Add checksumming to cfile .. WIP: KUDU-463. Add

[kudu-CR] log block manager: refactor container record processing

David Ribeiro Alves has posted comments on this change. Change subject: log_block_manager: refactor container record processing .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/6769 To unsubscrib

[kudu-CR] [KUDU-754] add an environment variable for kudu client debugging to stderr

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6736 to look at the new patch set (#3). Change subject: [KUDU-754] add an environment variable for kudu client debugging to stderr .

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Will Berkeley has posted comments on this change. Change subject: [webui] Fix master reg formatting and add memory use percentage. .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/6780/1/src/kudu/server/default-

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Will Berkeley has uploaded a new patch set (#2). Change subject: [webui] Fix master reg formatting and add memory use percentage. .. [webui] Fix master reg formatting and add memory use percentage. This is a little patch to fix

[kudu-CR] [webui] Fix master reg formatting and add memory use percentage.

Will Berkeley has uploaded a new change for review. http://gerrit.cloudera.org:8080/6780 Change subject: [webui] Fix master reg formatting and add memory use percentage. .. [webui] Fix master reg formatting and add memory use

[kudu-CR] env: add ReadV() API

Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6779 Change subject: env: add ReadV() API .. env: add ReadV() API Adds ReadV() methods to RWFile and RandomAccessFile that allows reading data

[kudu-CR] [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config

Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6772 to look at the new patch set (#4). Change subject: [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config

[kudu-CR] [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config

Will Berkeley has posted comments on this change. Change subject: [WIP] KUDU-1860: ksck doesn't identify tablets that are evicted but still in config .. Patch Set 1: (8 comments) I implemented a GetAllConsensusStates RPC and

[kudu-CR] [java client] improve flaky test ITClient

Jean-Daniel Cryans has posted comments on this change. Change subject: [java client] improve flaky test ITClient .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/6776/2/java/kudu-client/src/test/java/org/apache/

[kudu-CR] WIP: Expose row format flags in KuduScanner

David Ribeiro Alves has posted comments on this change. Change subject: WIP: Expose row format flags in KuduScanner .. Patch Set 6: Verified+1 Unrelated java failure -- To view, visit http://gerrit.cloudera.org:8080/6624 To u

[kudu-CR] WIP: Expose row format flags in KuduScanner

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 (#6). Change subject: WIP: Expose row format flags in KuduScanner .. WIP: Expose row f