[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Adar Dembo (Code Review)
Adar Dembo has submitted this change and it was merged. Change subject: block manager: start using the file cache .. block manager: start using the file cache This commit integrates the file cache into both the file and log blo

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5147 To unsubscribe, visit http://gerri

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5147 to look at the new patch set (#10). Change subject: block manager: start using the file cache .. block manager: sta

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 9: (2 comments) http://gerrit.cloudera.org:8080/#/c/5147/9/src/kudu/fs/block_manager.cc File src/kudu/fs/blo

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 9: (2 comments) http://gerrit.cloudera.org:8080/#/c/5147/9/src/kudu/fs/block_manager.cc File src/kudu/fs/bl

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/5147/4/src/kudu/fs/block_manager.cc File src/kudu/fs/blo

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5147 to look at the new patch set (#9). Change subject: block manager: start using the file cache .. block manager: star

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 8: (2 comments) http://gerrit.cloudera.org:8080/#/c/5147/4/src/kudu/fs/block_manager.cc File src/kudu/fs/bl

[kudu-CR] block manager: start using the file cache

2016-12-07 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5147 to look at the new patch set (#8). Change subject: block manager: start using the file cache .

[kudu-CR] block manager: start using the file cache

2016-12-06 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5147 to look at the new patch set (#7). Change subject: block manager: start using the file cache .. block manager: star

[kudu-CR] block manager: start using the file cache

2016-12-06 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5147/4/src/kudu/fs/block_manager.cc File src/kudu/fs/bloc

[kudu-CR] block manager: start using the file cache

2016-12-05 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 6: (1 comment) http://gerrit.cloudera.org:8080/#/c/5147/4/src/kudu/fs/block_manager.cc File src/kudu/fs/blo

[kudu-CR] block manager: start using the file cache

2016-12-01 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5147 to look at the new patch set (#5). Change subject: block manager: start using the file cache .

[kudu-CR] block manager: start using the file cache

2016-12-01 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5147/4/src/kudu/fs/log_block_manager.cc File src/kudu/fs/

[kudu-CR] block manager: start using the file cache

2016-11-30 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 4: (1 comment) http://gerrit.cloudera.org:8080/#/c/5147/4/src/kudu/fs/log_block_manager.cc File src/kudu/fs

[kudu-CR] block manager: start using the file cache

2016-11-22 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 4: > * Make sure this handles the case where the ulimit is unlimited As we discovered, it doesn't appear pos

[kudu-CR] block manager: start using the file cache

2016-11-22 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5147 to look at the new patch set (#4). Change subject: block manager: start using the file cache .. block manager: star

[kudu-CR] block manager: start using the file cache

2016-11-22 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 3: code looks good, going to jot down some notes we discussed offline: * Make sure this handles the case wh

[kudu-CR] block manager: start using the file cache

2016-11-21 Thread Adar Dembo (Code Review)
Adar Dembo has uploaded a new patch set (#3). Change subject: block manager: start using the file cache .. block manager: start using the file cache This commit integrates the file cache into both the file and log block managers

[kudu-CR] block manager: start using the file cache

2016-11-18 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: block manager: start using the file cache .. Patch Set 2: Verified+1 TSAN failure was KUDU-1624. -- To view, visit http://gerrit.cloudera.org:8080/5147 To unsubsc

[kudu-CR] block manager: start using the file cache

2016-11-18 Thread Adar Dembo (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5147 to look at the new patch set (#2). Change subject: block manager: start using the file cache .. block manager: star

[kudu-CR] block manager: start using the file cache

2016-11-18 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5147 to review the following change. Change subject: block manager: start using the file cache .. block manage