[kudu-CR] Give more context on errors reading cfiles

2017-08-23 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Give more context on errors reading cfiles .. Give more context on errors reading cfiles Recently, a user reported an error loading a tablet in which the

[kudu-CR] Give more context on errors reading cfiles

2017-08-23 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7620 To unsubscribe, visit

[kudu-CR] Give more context on errors reading cfiles

2017-08-23 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Andrew Wong, Grant Henke, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7620 to look at the new patch set (#2). Change subject: Give more context on errors reading cfiles

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: Code-Review+1 (1 comment) http://gerrit.cloudera.org:8080/#/c/7620/1/src/kudu/cfile/cfile_reader.cc

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: Code-Review+2 sounds good -- To view, visit http://gerrit.cloudera.org:8080/7620 To

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7620/1/src/kudu/cfile/cfile_reader.cc File

[kudu-CR] Give more context on errors reading cfiles

2017-08-10 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7620/1/src/kudu/cfile/cfile_reader.cc File

[kudu-CR] Give more context on errors reading cfiles

2017-08-08 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. Change subject: Give more context on errors reading cfiles .. Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/7620/1/src/kudu/cfile/cfile-test.cc File

[kudu-CR] Give more context on errors reading cfiles

2017-08-08 Thread Todd Lipcon (Code Review)
Hello David Ribeiro Alves, Grant Henke, Andrew Wong, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7620 to review the following change. Change subject: Give more context on errors reading cfiles