[kudu-CR](branch-1.4.x) * two improvements: 1) support open-ended intervals; 2) end up fetching one more rowset for the upper bound which is exclusive;

2018-07-10 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10917 Change subject: * two improvements: 1) support open-ended intervals; 2) end up fetching one more rowset for the upper bound which is exclusive; ...

[kudu-CR](branch-1.4.x) * two improvements: 1) support open-ended intervals; 2) end up fetching one more rowset for the upper bound which is exclusive;

2018-07-10 Thread helifu (Code Review)
helifu has abandoned this change. ( http://gerrit.cloudera.org:8080/10917 ) Change subject: * two improvements: 1) support open-ended intervals; 2) end up fetching one more rowset for the upper bound which is exclusive; .. Aban

[kudu-CR] KUDU-2529 add a "-tables=" flag to the "kudu table list".

2018-08-30 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11360 Change subject: KUDU-2529 add a "-tables=" flag to the "kudu table list". .. KUDU-2529 add a "-tables=" flag to the "kudu table list". Change

[kudu-CR] KUDU-2529: Add a "-tables=" flag to the "kudu table list".

2018-08-31 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11373 Change subject: KUDU-2529: Add a "-tables=" flag to the "kudu table list". .. KUDU-2529: Add a "-tables=" flag to the "kudu table list". "-ta

[kudu-CR] KUDU-2529 add a "-tables=" flag to the "kudu table list".

2018-08-31 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11360 to look at the new patch set (#2). Change subject: KUDU-2529 add a "-tables=" flag to the "kudu table list". ..

[kudu-CR] KUDU-2529 add a "-tables=" flag to the "kudu table list".

2018-09-02 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11360 to look at the new patch set (#3). Change subject: KUDU-2529 add a "-tables=" flag to the "kudu table list". ..

[kudu-CR] KUDU-2529: Add a "-tables=" flag to the "kudu table list".

2018-09-02 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11360 to look at the new patch set (#4). Change subject: KUDU-2529: Add a "-tables=" flag to the "kudu table list". .

[kudu-CR] KUDU-2529: Add a "-tables=" flag to the "kudu table list".

2018-09-02 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11360 to look at the new patch set (#5). Change subject: KUDU-2529: Add a "-tables=" flag to the "kudu table list". .

[kudu-CR] KUDU-2566:

2018-09-03 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11381 Change subject: KUDU-2566: .. KUDU-2566: two improvements: 1) support open-ended intervals; 2) end up fetching one more rowset for the upper b

[kudu-CR] KUDU-2566: fix two todo list

2018-09-04 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11381 to look at the new patch set (#2). Change subject: KUDU-2566: fix two todo list .. KUDU-2566: fix two tod

[kudu-CR] KUDU-2529: Add a "-tables=" flag to the "kudu table list".

2018-09-04 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11360 to look at the new patch set (#6). Change subject: KUDU-2529: Add a "-tables=" flag to the "kudu table list". .

[kudu-CR] KUDU-2529: Add a "-tables=" flag to the "kudu table list".

2018-09-04 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11360 to look at the new patch set (#7). Change subject: KUDU-2529: Add a "-tables=" flag to the "kudu table list". ...

[kudu-CR] KUDU-2529: Add a "-tables=" flag to the "kudu table list".

2018-09-05 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11360 to look at the new patch set (#8). Change subject: KUDU-2529: Add a "-tables=" flag to the "kudu table list". ...

[kudu-CR] KUDU-2566: Enhance rowset tree pruning and discard string copying while querying

2018-09-06 Thread helifu (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11381 to look at the new patch set (#3). Change subject: KUDU-2566: Enhance rowset tree pruning and discard string copying while querying ..

[kudu-CR] KUDU-2566: Enhance rowset tree pruning and discard string copying while querying

2018-09-07 Thread helifu (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11381 to look at the new patch set (#4). Change subject: KUDU-2566: Enhance rowset tree pruning and discard string copying while querying ..

[kudu-CR] KUDU-2566: Enhance rowset tree pruning and discard string copying while querying

2018-09-07 Thread helifu (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11381 to look at the new patch set (#5). Change subject: KUDU-2566: Enhance rowset tree pruning and discard string copying while querying ..

[kudu-CR] KUDU-2566: Enhance rowset tree pruning and stop coping string while querying

2018-09-09 Thread helifu (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11381 to look at the new patch set (#6). Change subject: KUDU-2566: Enhance rowset tree pruning and stop coping string while querying ...

[kudu-CR] KUDU-2566: Enhance rowset tree pruning and stop copying strings

2018-09-10 Thread helifu (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11381 to look at the new patch set (#7). Change subject: KUDU-2566: Enhance rowset tree pruning and stop copying strings

[kudu-CR] KUDU-2566: Enhance rowset tree pruning and stop copying strings

2018-09-10 Thread helifu (Code Review)
Hello Will Berkeley, Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11381 to look at the new patch set (#8). Change subject: KUDU-2566: Enhance rowset tree pruning and stop copying strings

[kudu-CR] KUDU-2038: Support bitmap indexing

2018-10-18 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/11722 Change subject: KUDU-2038: Support bitmap indexing .. KUDU-2038: Support bitmap indexing One DRS has a set of bitmap info independently, one i

[kudu-CR] KUDU-2038: Support bitmap indexing

2018-10-19 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11722 to look at the new patch set (#2). Change subject: KUDU-2038: Support bitmap indexing .. KUD

[kudu-CR] KUDU-2038: Support bitmap indexing

2018-10-31 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/11722 ) Change subject: KUDU-2038: Support bitmap indexing .. Patch Set 2: (6 comments) http://gerrit.cloudera.org:8080/#/c/11722/1//COMMIT_MSG Comm

[kudu-CR] KUDU-2529: Add a "-tables=" flag to the "kudu table list".

2018-10-31 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/11360 ) Change subject: KUDU-2529: Add a "-tables=" flag to the "kudu table list". .. Patch Set 9: (17 comments) sorry for the late reply, i though

[kudu-CR] KUDU-2566: Enhance rowset tree pruning and stop copying strings

2018-10-31 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/11381 ) Change subject: KUDU-2566: Enhance rowset tree pruning and stop copying strings .. Patch Set 9: (47 comments) thanks to Adar and Will. http

[kudu-CR] KUDU-2038: Support bitmap index

2018-11-13 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11722 to look at the new patch set (#3). Change subject: KUDU-2038: Support bitmap index .. KUDU-2

[kudu-CR] KUDU-2038: Support bitmap index

2018-11-13 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11722 to look at the new patch set (#4). Change subject: KUDU-2038: Support bitmap index .. KUDU-2

[kudu-CR] KUDU-2038: Support bitmap index

2018-11-13 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11722 to look at the new patch set (#5). Change subject: KUDU-2038: Support bitmap index .. KUDU-2

[kudu-CR] KUDU-2038: Support bitmap index

2018-11-13 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/11722 to look at the new patch set (#6). Change subject: KUDU-2038: Support bitmap index .. KUDU-2

[kudu-CR] KUDU-2038: Support bitmap index

2018-11-13 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/11722 ) Change subject: KUDU-2038: Support bitmap index .. Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/11722/1//COMMIT_MSG Commit

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-12 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12075 Change subject: KUDU-2636: LBM supports deleting dead and full containers .. KUDU-2636: LBM supports deleting dead and full containers LBM sup

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-12 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 1: (3 comments) Thanks for your comments.@adar http:

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-12 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#2). Change subject: KUDU-2636: LBM supports deleting dead and full containers ...

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-13 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/12

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-13 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#3). Change subject: KUDU-2636: LBM supports deleting dead and full containers ...

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-17 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 3: (11 comments) thanks for your detailed comments.

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-17 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#4). Change subject: KUDU-2636: LBM supports deleting dead and full containers ..

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-17 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 4: (6 comments) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-18 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/12

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-18 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 4: (2 comments) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-18 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#5). Change subject: KUDU-2636: LBM supports deleting dead and full containers ..

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-18 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 5: (5 comments) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-18 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#6). Change subject: KUDU-2636: LBM supports deleting dead and full containers ..

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-19 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 6: (11 comments) There are more than one LogBlock whi

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-19 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 6: maybe that is not enough. please take a look at thi

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-21 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#7). Change subject: KUDU-2636: LBM supports deleting dead and full containers ..

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2018-12-22 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12121 Change subject: KUDU-2636: LBM supports deleting dead and full containers .. KUDU-2636: LBM supports deleting dead and full containers LBM sup

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2018-12-26 Thread helifu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12121 to look at the new patch set (#2). Change subject: [fs]: wrapping up containers in scoped_refptr .. [fs]: wrapping

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2018-12-26 Thread helifu (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12121 to look at the new patch set (#3). Change subject: [fs]: wrapping up containers in scoped_refptr .. [fs]: wrapping

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2018-12-27 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12121 ) Change subject: [fs]: wrapping up containers in scoped_refptr .. Patch Set 3: I compiled the kudu code on debian 8.9 locally, and the unit te

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2019-01-04 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12121 ) Change subject: [fs]: wrapping up containers in scoped_refptr .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/12121/3/src/k

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2019-01-04 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12121 to look at the new patch set (#4). Change subject: [fs]: wrapping up containers in scoped_refptr .. [fs

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2019-01-04 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12121 ) Change subject: [fs]: wrapping up containers in scoped_refptr .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/12121/4/src/ku

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-05 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#8). Change subject: KUDU-2636: LBM supports deleting dead and full containers ..

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2019-01-07 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12121 ) Change subject: [fs]: wrapping up containers in scoped_refptr .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/12121/4/src/ku

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2019-01-07 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12121 to look at the new patch set (#5). Change subject: [fs]: wrapping up containers in scoped_refptr .

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-07 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#10). Change subject: KUDU-2636: LBM supports deleting dead and full containers ...

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-08 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 11: (3 comments) Thank you very much :) And here are

[kudu-CR] [fs]: wrapping up containers in scoped refptr

2019-01-08 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12121 ) Change subject: [fs]: wrapping up containers in scoped_refptr .. Patch Set 6: Here is a tserver node of our online cluster, and it has been r

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 11: All right, calling 'SyncDir()' has been removed.

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#12). Change subject: KUDU-2636: LBM supports deleting dead and full containers ...

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 12: (6 comments) http://gerrit.cloudera.org:8080/#/c/

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-09 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#13). Change subject: KUDU-2636: LBM supports deleting dead and full containers ...

[kudu-CR] KUDU-2636: LBM supports deleting dead and full containers

2019-01-10 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead and full containers .. Patch Set 13: Wow, ASAN fails again. -- To view, visit http://

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-10 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#14). Change subject: KUDU-2636: LBM supports deleting dead containers

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-10 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#15). Change subject: KUDU-2636: LBM supports deleting dead containers

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-10 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead containers .. Patch Set 14: (2 comments) http://gerrit.cloudera.org:8080/#/c/12075/12/

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead containers .. Patch Set 15: I'm sorry to have omitted this case! :( -- To view, visit

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#16). Change subject: KUDU-2636: LBM supports deleting dead containers

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead containers .. Patch Set 15: All the tests passed locally except ASAN. Failing on an unr

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead containers .. Patch Set 16: (2 comments) http://gerrit.cloudera.org:8080/#/c/12075/12/

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#17). Change subject: KUDU-2636: LBM supports deleting dead containers

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12075 ) Change subject: KUDU-2636: LBM supports deleting dead containers .. Patch Set 17: (1 comment) http://gerrit.cloudera.org:8080/#/c/12075/17/s

[kudu-CR] KUDU-2636: LBM supports deleting dead containers

2019-01-11 Thread helifu (Code Review)
Hello Tidy Bot, Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12075 to look at the new patch set (#18). Change subject: KUDU-2636: LBM supports deleting dead containers

[kudu-CR] [fs]: LBM moves half-present containers out of the way

2019-01-18 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12239 Change subject: [fs]: LBM moves half-present containers out of the way .. [fs]: LBM moves half-present containers out of the way this patch in

[kudu-CR] [fs]: LBM moves half-present containers out of the way

2019-01-20 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: LBM moves half-present containers out of the way .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/1223

[kudu-CR] [fs]: LBM moves half-present containers out of the way

2019-01-20 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: LBM moves half-present containers out of the way .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12239

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-21 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12239 to look at the new patch set (#2). Change subject: [fs]: delete the (orphaned) metadata file when repairing

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-21 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12239 to look at the new patch set (#3). Change subject: [fs]: delete the (orphaned) metadata file when repairing

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12254/1/src/kud

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12254/1/src/kud

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: my preference 2) is not safe because we can't be sure when the t

[kudu-CR] KUDU-2665: deflake block manager-stress-test

2019-01-22 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12254 ) Change subject: KUDU-2665: deflake block_manager-stress-test .. Patch Set 1: Code-Review+1 Reread the ThreadPool code, it seems that 'dd_mana

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-22 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12239 to look at the new patch set (#4). Change subject: [fs]: delete the (orphaned) metadata file when repairing ...

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-23 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 4: (12 comments) http://gerrit.cloudera.org:8080/#/c/1

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-23 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/122

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-23 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12239 to look at the new patch set (#5). Change subject: [fs]: delete the (orphaned) metadata file when repairing ...

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-23 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 5: (2 comments) http://gerrit.cloudera.org:8080/#/c/12

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-23 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12239 to look at the new patch set (#6). Change subject: [fs]: delete the (orphaned) metadata file when repairing ...

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-24 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12239 ) Change subject: [fs]: delete the (orphaned) metadata file when repairing .. Patch Set 6: (5 comments) http://gerrit.cloudera.org:8080/#/c/12

[kudu-CR] [fs]: delete the (orphaned) metadata file when repairing

2019-01-24 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12239 to look at the new patch set (#7). Change subject: [fs]: delete the (orphaned) metadata file when repairing ...

[kudu-CR] Fix compilation warnings under debian8.9

2019-01-29 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12294 Change subject: Fix compilation warnings under debian8.9 .. Fix compilation warnings under debian8.9 It seems there are some compilation warni

[kudu-CR] KUDU2665: LBM deletes the valid live block

2019-01-30 Thread helifu (Code Review)
helifu has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12308 Change subject: KUDU2665: LBM deletes the valid live block .. KUDU2665: LBM deletes the valid live block Due to the nature of WritableBlock fi

[kudu-CR] KUDU2665: LBM deletes the valid live block

2019-01-30 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12308 ) Change subject: KUDU2665: LBM deletes the valid live block .. Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/12308/1/src/kudu

[kudu-CR] KUDU2665: LBM may delete containers with live blocks

2019-01-30 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12308 to look at the new patch set (#2). Change subject: KUDU2665: LBM may delete containers with live blocks

[kudu-CR] KUDU2665: LBM may delete containers with live blocks

2019-01-30 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12308 ) Change subject: KUDU2665: LBM may delete containers with live blocks .. Patch Set 2: if we comment 'blocks_being_written() == 0' in the log_b

[kudu-CR] KUDU2665: LBM may delete containers with live blocks

2019-01-31 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12308 ) Change subject: KUDU2665: LBM may delete containers with live blocks .. Patch Set 2: (5 comments) http://gerrit.cloudera.org:8080/#/c/12308/

[kudu-CR] KUDU2665: LBM may delete containers with live blocks

2019-01-31 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12308 ) Change subject: KUDU2665: LBM may delete containers with live blocks .. Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/12308/

[kudu-CR] KUDU-2665: LBM may delete containers with live blocks

2019-01-31 Thread helifu (Code Review)
Hello Kudu Jenkins, Adar Dembo, Hao Hao, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12308 to look at the new patch set (#3). Change subject: KUDU-2665: LBM may delete containers with live blocks ..

[kudu-CR] Fix compilation warnings under debian8.9

2019-01-31 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12294 ) Change subject: Fix compilation warnings under debian8.9 .. Patch Set 1: (9 comments) http://gerrit.cloudera.org:8080/#/c/12294/1/src/kudu/c

[kudu-CR] KUDU-2665: LBM may delete containers with live blocks

2019-02-01 Thread helifu (Code Review)
helifu has posted comments on this change. ( http://gerrit.cloudera.org:8080/12308 ) Change subject: KUDU-2665: LBM may delete containers with live blocks .. Patch Set 3: (4 comments) http://gerrit.cloudera.org:8080/#/c/12308

  1   2   3   4   5   >