[kudu-CR] KUDU-2238. DMS not flush under memory pressure.

2018-01-16 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8904 ) Change subject: KUDU-2238. DMS not flush under memory pressure. .. Patch Set 4: (1 comment)

[kudu-CR] KUDU-2238. DMS not flush under memory pressure.

2018-01-16 Thread zhen.zhang (Code Review)
Hello Mike Percy, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8904 to look at the new patch set (#4). Change subject: KUDU-2238. DMS not flush under memory pressure.

[kudu-CR] KUDU-2238. DMS not flush under memory pressure.

2018-01-16 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8904 ) Change subject: KUDU-2238. DMS not flush under memory pressure. .. Patch Set 3: (1 comment)

[kudu-CR] KUDU-2261: The order of the responses after flush should match the order we call apply

2018-01-16 Thread zhen.zhang (Code Review)
Hello Mike Percy, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/9029 to look at the new patch set (#2). Change subject: KUDU-2261: The order of the responses after flush should match the order we call apply

[kudu-CR] KUDU-2261: The order of the responses after flush should match the order we call apply

2018-01-16 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/9029 ) Change subject: KUDU-2261: The order of the responses after flush should match the order we call apply .. Patch Set 1: The test failed

[kudu-CR] KUDU-2261: The order of the responses after flush should match the order we call apply

2018-01-16 Thread zhen.zhang (Code Review)
zhen.zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9029 Change subject: KUDU-2261: The order of the responses after flush should match the order we call apply .. KUDU-2261: The order of the

[kudu-CR] KUDU-2238. DMS not flush under memory pressure.

2017-12-22 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. ( http://gerrit.cloudera.org:8080/8904 ) Change subject: KUDU-2238. DMS not flush under memory pressure. .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2238. DMS not flush under memory pressure.

2017-12-22 Thread zhen.zhang (Code Review)
Hello Mike Percy, Jean-Daniel Cryans, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8904 to look at the new patch set (#2). Change subject: KUDU-2238. DMS not flush under memory pressure.

[kudu-CR] KUDU-2238. DMS not flush under memory pressure.

2017-12-21 Thread zhen.zhang (Code Review)
zhen.zhang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/8904 Change subject: KUDU-2238. DMS not flush under memory pressure. .. KUDU-2238. DMS not flush under memory pressure. When we choose DMS to

[kudu-CR] Add KuduTable::formatted range partitions in cpp client.

2017-02-07 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5893 to look at the new patch set (#2). Change subject: Add KuduTable::formatted_range_partitions in cpp client. .. Add

[kudu-CR] Add KuduTable::formatted range partitions in cpp client.

2017-02-03 Thread zhen.zhang (Code Review)
zhen.zhang has uploaded a new change for review. http://gerrit.cloudera.org:8080/5893 Change subject: Add KuduTable::formatted_range_partitions in cpp client. .. Add KuduTable::formatted_range_partitions in cpp client. This

[kudu-CR] docs: add notes for multi master migration

2017-01-19 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5728 to look at the new patch set (#4). Change subject: docs: add notes for multi master migration .. docs: add notes

[kudu-CR] docs: add notes for multi master migration

2017-01-19 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. Change subject: docs: add notes for multi master migration .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5728/3/docs/administration.adoc File

[kudu-CR] docs: add notes for multi master migration

2017-01-19 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. Change subject: docs: add notes for multi master migration .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/5728/3/docs/administration.adoc File

[kudu-CR] docs: add notes for multi master migration

2017-01-19 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5728 to look at the new patch set (#3). Change subject: docs: add notes for multi master migration .. docs: add notes

[kudu-CR] docs: add notes for multi master migration

2017-01-19 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. Change subject: docs: add notes for multi master migration .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/5728/2/docs/administration.adoc File

[kudu-CR] docs: add notes for multi master migration

2017-01-18 Thread zhen.zhang (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5728 to look at the new patch set (#2). Change subject: docs: add notes for multi master migration .. docs: add notes

[kudu-CR] docs: add notes for multi master migration

2017-01-18 Thread zhen.zhang (Code Review)
zhen.zhang has posted comments on this change. Change subject: docs: add notes for multi master migration .. Patch Set 1: (2 comments) I just installed "Asciidoctor.js Live Preview", it's very useful, thanks :)