[kudu-CR] KUDU-1844: /varz should not expose potentially sensitive configs

2017-02-16 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6043 to look at the new patch set (#2). Change subject: KUDU-1844: /varz should not expose potentially sensitive configs

[kudu-CR] KUDU-1844: /varz should not expose potentially sensitive configs

2017-02-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1844: /varz should not expose potentially sensitive configs .. Patch Set 1: (24 comments) http://gerrit.cloudera.org:8080/#/c/6043/1//COMMIT_MSG Commit Message:

[kudu-CR] KUDU-1844: /varz should not expose potentially sensitive configs

2017-02-16 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6043 Change subject: KUDU-1844: /varz should not expose potentially sensitive configs .. KUDU-1844: /varz should not expose potentially sensitive

[kudu-CR] KUDU-1844: /varz should not expose potentially sensitive configs

2017-02-16 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1844: /varz should not expose potentially sensitive configs .. Patch Set 3: The failed test "MultiThreadedTabletTest/0.DeleteAndReinsert" seems to pass on my

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-22 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6112 Change subject: Combine redaction flag for log and flags into one. .. Combine redaction flag for log and flags into one. This adds a new flag

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-23 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#6). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-22 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#3). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-23 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#5). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-23 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#4). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-22 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2). Change subject: Combine redaction flag for log and flags into one. .. Combine redaction flag for log and flags into one. This adds a new flag --redact to replace

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Combine redaction flag for log and flags into one. .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/6112/11/src/kudu/util/flags.cc File

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#12). Change subject: Combine redaction flag for log and flags into one.

[kudu-CR] KUDU-1898: /varz page doesn't HTML-escape flag values

2017-02-27 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6173 Change subject: KUDU-1898: /varz page doesn't HTML-escape flag values .. KUDU-1898: /varz page doesn't HTML-escape flag values This fix the

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#11). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Combine redaction flag for log and flags into one. .. Patch Set 11: (1 comment) http://gerrit.cloudera.org:8080/#/c/6112/10/src/kudu/util/flags.cc File

[kudu-CR] KUDU-1898: /varz page doesn't HTML-escape flag values

2017-02-28 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1898: /varz page doesn't HTML-escape flag values .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6173/1//COMMIT_MSG Commit Message: PS1, Line

[kudu-CR] KUDU-1898: /varz page doesn't HTML-escape flag values

2017-02-28 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6173 to look at the new patch set (#3). Change subject: KUDU-1898: /varz page doesn't HTML-escape flag values ..

[kudu-CR] KUDU-1896 (part 1). Add redaction to JSON protobuf output

2017-02-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1896 (part 1). Add redaction to JSON protobuf output .. Patch Set 1: Code-Review+1 (1 comment)

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-26 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Combine redaction flag for log and flags into one. .. Patch Set 8: (11 comments) http://gerrit.cloudera.org:8080/#/c/6112/7/src/kudu/util/flags.cc File

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Combine redaction flag for log and flags into one. .. Patch Set 9: (6 comments) http://gerrit.cloudera.org:8080/#/c/6112/8/src/kudu/common/column_predicate-test.cc

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#9). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-26 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#8). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Combine redaction flag for log and flags into one. .. Patch Set 10: (1 comment) http://gerrit.cloudera.org:8080/#/c/6112/10/src/kudu/util/flags.cc File

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6112 to look at the new patch set (#10). Change subject: Combine redaction flag for log and flags into one. .. Combine

[kudu-CR] Combine redaction flag for log and flags into one.

2017-02-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Combine redaction flag for log and flags into one. .. Patch Set 9: (2 comments) http://gerrit.cloudera.org:8080/#/c/6112/9/src/kudu/util/flags.cc File

[kudu-CR] KUDU-1898: /varz page doesn't HTML-escape flag values

2017-02-28 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6173 to look at the new patch set (#4). Change subject: KUDU-1898: /varz page doesn't HTML-escape flag values ..

[kudu-CR] KUDU-1898: /varz page doesn't HTML-escape flag values

2017-02-28 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6173 to look at the new patch set (#7). Change subject: KUDU-1898: /varz page doesn't HTML-escape flag values ..

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-08 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4945 to look at the new patch set (#5). Change subject: Reject CREATE TABLE ops with even replication factor .. Reject

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Reject CREATE TABLE ops with even replication factor .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4945/2/src/kudu/master/catalog_manager.cc File

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-06 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4945 to look at the new patch set (#3). Change subject: Reject CREATE TABLE ops with even replication factor .. Reject

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-06 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. Change subject: Reject CREATE TABLE ops with even replication factor .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/4972 To unsubscribe, visit

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-06 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/4972 Change subject: Reject CREATE TABLE ops with even replication factor .. Reject CREATE TABLE ops with even replication factor Reject table

[kudu-CR] KUDU-1658: Reject CREATE TABLE ops with even replication factor

2016-11-22 Thread Hao Hao (Code Review)
Hello Dinesh Bhat, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4945 to look at the new patch set (#6). Change subject: KUDU-1658: Reject CREATE TABLE ops with even replication factor

[kudu-CR] KUDU-1658: Reject CREATE TABLE ops with even replication factor

2016-11-28 Thread Hao Hao (Code Review)
Hello Dinesh Bhat, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4945 to look at the new patch set (#7). Change subject: KUDU-1658: Reject CREATE TABLE ops with even replication factor

[kudu-CR] KUDU-1658: Reject CREATE TABLE ops with even replication factor

2016-11-28 Thread Hao Hao (Code Review)
Hello Dinesh Bhat, Alexey Serbin, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4945 to look at the new patch set (#8). Change subject: KUDU-1658: Reject CREATE TABLE ops with even replication factor

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-04 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2). Change subject: Reject CREATE TABLE ops with even replication factor .. Reject CREATE TABLE ops with even replication factor Reject table creation with even replication factor, and

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-04 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/4945 Change subject: Reject CREATE TABLE ops with even replication factor .. Reject CREATE TABLE ops with even replication factor Reject table

[kudu-CR] Reject CREATE TABLE ops with even replication factor

2016-11-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Reject CREATE TABLE ops with even replication factor .. Patch Set 2: Working on the fix.. -- To view, visit http://gerrit.cloudera.org:8080/4945 To unsubscribe,

[kudu-CR] [docs] Add security guide

2017-03-27 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [docs] Add security guide .. Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/6479/1/docs/security.adoc File docs/security.adoc: Line 31: access from

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

2017-03-30 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6514 Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs .. KUDU-1875: Refuse unauthenticated

[kudu-CR] [docs] Add security guide

2017-04-04 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [docs] Add security guide .. Patch Set 3: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6479 To unsubscribe, visit

[kudu-CR] [docs] Add security guide

2017-04-03 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [docs] Add security guide .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/6479/1/docs/security.adoc File docs/security.adoc: Line 110: connections.

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-05 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6566 to look at the new patch set (#2). Change subject: KUDU-579 [java_client] Scanner fault tolerance .. KUDU-579

[kudu-CR] WIP KUDU-579 [java client] Scanner fault tolerance

2017-04-20 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: WIP KUDU-579 [java_client] Scanner fault tolerance .. Patch Set 5: (4 comments)

[kudu-CR] WIP KUDU-579 [java client] Scanner fault tolerance

2017-04-20 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6701 Change subject: WIP KUDU-579 [java_client] Scanner fault tolerance .. WIP KUDU-579 [java_client] Scanner fault tolerance This patch adds java

[kudu-CR] WIP KUDU-579 [java client] Scanner fault tolerance

2017-04-20 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6566 to look at the new patch set (#5). Change subject: WIP KUDU-579 [java_client] Scanner fault tolerance .. WIP

[kudu-CR] WIP KUDU-579 [java client] Scanner fault tolerance

2017-04-20 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. Change subject: WIP KUDU-579 [java_client] Scanner fault tolerance .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/6701 To unsubscribe, visit

[kudu-CR] Fix Webserver option 'password file' to pass correct Squeasel option 'global auth file' for enabling HTTP authorization.

2017-03-08 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Fix Webserver option 'password_file' to pass correct Squeasel option 'global_auth_file' for enabling HTTP authorization. .. Patch Set 3: (2 comments)

[kudu-CR] webserver: add X-Frame-Options header

2017-03-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: webserver: add X-Frame-Options header .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/6215/3/src/kudu/server/webserver.cc File

[kudu-CR] Fix Webserver option 'password file' to pass correct Squeasel option 'global auth file' for enabling HTTP authorization.

2017-03-07 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6300 to look at the new patch set (#3). Change subject: Fix Webserver option 'password_file' to pass correct Squeasel option 'global_auth_file' for enabling HTTP authorization.

[kudu-CR] Fix Webserver option 'password file' to pass correct Squeasel option 'global auth file' for enabling HTTP authorization.

2017-03-07 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6300 Change subject: Fix Webserver option 'password_file' to pass correct Squeasel option 'global_auth_file' for enabling HTTP authorization. ..

[kudu-CR] Fix Webserver option 'password file' to pass correct Squeasel option 'global auth file' for enabling HTTP authorization.

2017-03-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Fix Webserver option 'password_file' to pass correct Squeasel option 'global_auth_file' for enabling HTTP authorization. .. Patch Set 1: (2 comments)

[kudu-CR] Fix Webserver option 'password file' to pass correct Squeasel option 'global auth file' for enabling HTTP authorization.

2017-03-07 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2). Change subject: Fix Webserver option 'password_file' to pass correct Squeasel option 'global_auth_file' for enabling HTTP authorization. .. Fix Webserver option 'password_file' to pass

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-579 [java_client] Scanner fault tolerance .. Patch Set 3: (3 comments)

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-07 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6566 to look at the new patch set (#4). Change subject: KUDU-579 [java_client] Scanner fault tolerance .. KUDU-579

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 4: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/6590 To unsubscribe, visit

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-579 [java_client] Scanner fault tolerance .. Patch Set 3: (4 comments)

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-05 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/6566 Change subject: KUDU-579 [java_client] Scanner fault tolerance .. KUDU-579 [java_client] Scanner fault tolerance This patch adds java client

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-06 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-579 [java_client] Scanner fault tolerance .. Patch Set 3: (17 comments)

[kudu-CR] KUDU-579 [java client] Scanner fault tolerance

2017-04-06 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6566 to look at the new patch set (#3). Change subject: KUDU-579 [java_client] Scanner fault tolerance .. KUDU-579

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

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs .. Patch Set 3: (3 comments)

[kudu-CR] Add --redact=none flag option for specifying no redaction

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: Add --redact=none flag option for specifying no redaction .. Patch Set 3: The change looks good to me, but can you check why the tests are failing? It looks like

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

2017-04-07 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6514 to look at the new patch set (#5). Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs

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

2017-04-07 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6514 to look at the new patch set (#4). Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs

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

2017-04-07 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1875: Refuse unauthenticated connections from publicly routable IP addrs .. Patch Set 4: (5 comments)

[kudu-CR](branch-1.3.x) [docs] Add admin workflow for recovering from disk failure

2017-04-18 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [docs] Add admin workflow for recovering from disk failure .. Patch Set 1: Code-Review+1 (1 comment)

[kudu-CR] WIP KUDU-1943: Add BlockTransaction to Block Manager

2017-08-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: WIP KUDU-1943: Add BlockTransaction to Block Manager .. Patch Set 8: (41 comments) http://gerrit.cloudera.org:8080/#/c/7207/8/src/kudu/cfile/bloomfile.h File

[kudu-CR] WIP KUDU-1943: Add BlockTransaction to Block Manager

2017-08-02 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#9). Change subject: WIP KUDU-1943: Add BlockTransaction to Block Manager .. WIP

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-02 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. Patch Set 10: (3 comments) http://gerrit.cloudera.org:8080/#/c/7207/9/src/kudu/cfile/cfile-test.cc File

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-02 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#10). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-04 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#11). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-15 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#16). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] [WIP] Add BlockDeletionTransaction to Block Manager

2017-08-10 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7656 Change subject: [WIP] Add BlockDeletionTransaction to Block Manager .. [WIP] Add BlockDeletionTransaction to Block Manager Similar to

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-10 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#14). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-10 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. Patch Set 14: (17 comments) http://gerrit.cloudera.org:8080/#/c/7207/13//COMMIT_MSG Commit Message: PS13, Line

[kudu-CR] [WIP] Add BlockDeletionTransaction to Block Manager

2017-08-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP] Add BlockDeletionTransaction to Block Manager .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/7656/1/src/kudu/fs/file_block_manager.cc File

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-11 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#15). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. Patch Set 15: (3 comments) http://gerrit.cloudera.org:8080/#/c/7207/14/src/kudu/fs/block_manager-stress-test.cc

[kudu-CR] log block manager: Reorder class declaration of LogWritableBlock

2017-08-11 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7595 to look at the new patch set (#3). Change subject: log block manager: Reorder class declaration of LogWritableBlock

[kudu-CR] log block manager: Reorder class declaration of LogWritableBlock

2017-08-11 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: log block manager: Reorder class declaration of LogWritableBlock .. Patch Set 3: (1 comment)

[kudu-CR] [WIP] Add BlockDeletionTransaction to Block Manager

2017-08-11 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7656 to look at the new patch set (#2). Change subject: [WIP] Add BlockDeletionTransaction to Block Manager .. [WIP]

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

2017-08-17 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7700 Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. KUDU-1726: Avoid fsync-per-block in tablet copy This patch incorporates

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-17 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#19). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

2017-08-17 Thread Hao Hao (Code Review)
Hao Hao has abandoned this change. Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. Abandoned -- To view, visit http://gerrit.cloudera.org:8080/7700 To unsubscribe, visit

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

2017-08-17 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new patch set (#2). Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. KUDU-1726: Avoid fsync-per-block in tablet copy This patch incorporates BlockTransaction API with tablet copy, to

[kudu-CR] KUDU-1726: Avoid fsync-per-block in tablet copy

2017-08-17 Thread Hao Hao (Code Review)
Hao Hao has uploaded a new change for review. http://gerrit.cloudera.org:8080/7701 Change subject: KUDU-1726: Avoid fsync-per-block in tablet copy .. KUDU-1726: Avoid fsync-per-block in tablet copy This patch incorporates

[kudu-CR] KUDU-1943: Add BlockTransaction to Block Manager

2017-08-16 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7207 to look at the new patch set (#18). Change subject: KUDU-1943: Add BlockTransaction to Block Manager .. KUDU-1943:

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-17 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager:update container in-memory accounting after data is synced .. Patch Set 5: Hi Dan, the test is not always guaranteed to fail but should have

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#5). Change subject: [WIP]log block manager:update container in-memory accounting after data is synced

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#3). Change subject: [WIP]log block manager:update container in-memory accounting after data is synced

[kudu-CR] [WIP]log block manager:update container in-memory accounting after data is synced

2017-07-07 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#4). Change subject: [WIP]log block manager:update container in-memory accounting after data is synced

[kudu-CR] [WIP]log block manager:mark container as 'read only' after syncing error

2017-07-21 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#8). Change subject: [WIP]log block manager:mark container as 'read_only' after syncing error

[kudu-CR] [WIP]log block manager:mark container as 'read only' after syncing error

2017-07-21 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager:mark container as 'read_only' after syncing error .. Patch Set 8: (12 comments)

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#11). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 12: The failure spark test does not seem to be relate. -- To view, visit

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-25 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#12). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 9: (11 comments)

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#9). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7374 to look at the new patch set (#10). Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error

[kudu-CR] [WIP]log block manager: mark container as 'read only' after syncing error

2017-07-24 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. Change subject: [WIP]log block manager: mark container as 'read_only' after syncing error .. Patch Set 10: (1 comment)

  1   2   3   4   5   6   7   8   9   10   >