[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-18 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 3: It would be good to provide a working code sample

[kudu-CR] KUDU-2456 Limit number of pending transactions logged

2019-01-18 Thread Bankim Bhavsar (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12240 to look at the new patch set (#2). Change subject: KUDU-2456 Limit number of pending transactions logged

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

2019-01-18 Thread Adar Dembo (Code Review)
Adar Dembo 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)

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

2019-01-18 Thread Adar Dembo (Code Review)
Adar Dembo 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)

[kudu-CR] KUDU-2456 Limit number of pending transactions logged

2019-01-18 Thread Greg Solovyev (Code Review)
Greg Solovyev has posted comments on this change. ( http://gerrit.cloudera.org:8080/12240 ) Change subject: KUDU-2456 Limit number of pending transactions logged .. Patch Set 1: (1 comment)

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-18 Thread Alexey Serbin (Code Review)
Hello Will Berkeley, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12119 to look at the new patch set (#4). Change subject: [blog] a blogpost about location awareness in Kudu

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 3: (1 comment) > (1 comment) > > > We have this

[kudu-CR] Various mostly cosmetic improvements to the maintenance manager

2019-01-18 Thread Will Berkeley (Code Review)
Will Berkeley has uploaded this change for review. ( http://gerrit.cloudera.org:8080/12246 Change subject: Various mostly cosmetic improvements to the maintenance manager .. Various mostly cosmetic improvements to the

[kudu-CR] Various mostly cosmetic improvements to the maintenance manager

2019-01-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12246 ) Change subject: Various mostly cosmetic improvements to the maintenance manager .. Patch Set 1: Code-Review+2 -- To view, visit

[kudu-CR] [examples] a small update on SparkExample

2019-01-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12208 ) Change subject: [examples] a small update on SparkExample .. Patch Set 3: (1 comment)

[kudu-CR] [java] throw exception if applying operation on a closed session

2019-01-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12237 ) Change subject: [java] throw exception if applying operation on a closed session .. Patch Set 4: (1 comment)

[kudu-CR] Various mostly cosmetic improvements to the maintenance manager

2019-01-18 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12246 ) Change subject: Various mostly cosmetic improvements to the maintenance manager .. Various mostly cosmetic improvements to the

[kudu-CR] Reduce DMS flush logging

2019-01-18 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12235 ) Change subject: Reduce DMS flush logging .. Reduce DMS flush logging Just like delta compactions, DMS flush logging can get

[kudu-CR] Reduce MRS flush and rowset compaction logging

2019-01-18 Thread Will Berkeley (Code Review)
Will Berkeley has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12234 ) Change subject: Reduce MRS flush and rowset compaction logging .. Reduce MRS flush and rowset compaction logging MRS flushes

[kudu-CR] KUDU-2456 Limit number of pending transactions logged

2019-01-18 Thread Bankim Bhavsar (Code Review)
Hello Alexey Serbin, Kudu Jenkins, Adar Dembo, Greg Solovyev, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/12240 to look at the new patch set (#3). Change subject: KUDU-2456 Limit number of pending transactions logged

[kudu-CR] Reduce DMS flush logging

2019-01-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12235 ) Change subject: Reduce DMS flush logging .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/12235 To

[kudu-CR] Various mostly cosmetic improvements to the maintenance manager

2019-01-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12246 ) Change subject: Various mostly cosmetic improvements to the maintenance manager .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] KUDU-2456 Limit number of pending transactions logged

2019-01-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12240 ) Change subject: KUDU-2456 Limit number of pending transactions logged .. Patch Set 3: (3 comments)

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-18 Thread Will Berkeley (Code Review)
Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 3: (1 comment) > We have this script which we use

[kudu-CR] Reduce MRS flush and rowset compaction logging

2019-01-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12234 ) Change subject: Reduce MRS flush and rowset compaction logging .. Patch Set 2: Code-Review+2 -- To view, visit

[kudu-CR] [examples] a small update on SparkExample

2019-01-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12208 ) Change subject: [examples] a small update on SparkExample .. [examples] a small update on SparkExample Updated the Spark

[kudu-CR](gh-pages) [blog] a blogpost about location awareness in Kudu

2019-01-18 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/12119 ) Change subject: [blog] a blogpost about location awareness in Kudu .. Patch Set 3: > It would be good to provide a working code

[kudu-CR] KUDU-2456 Limit number of pending transactions logged

2019-01-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/12240 ) Change subject: KUDU-2456 Limit number of pending transactions logged .. Patch Set 3: BTW, one of our code review norms is to respond to