Todd Lipcon has posted comments on this change. Change subject: KUDU-1396. Fix index building when an individual value overflows block ......................................................................
Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/2725/2/src/kudu/cfile/cfile-test-base.h File src/kudu/cfile/cfile-test-base.h: Line 246: std::vector<std::string> data_buffers_; > Just curious: the motivation for this change was so you could store strings Yes, the test stores strings that are 65KB http://gerrit.cloudera.org:8080/#/c/2725/2/src/kudu/cfile/cfile-test.cc File src/kudu/cfile/cfile-test.cc: Line 504: ASSERT_EQ(StringPrintf(str_format, 5000), s.ToString()); > May be clearer if, before each StringPrintf (or loop that does the equivale Will do Line 524: arena.Reset(); > What happens if the arena isn't reset, as before? It just uses a lot more memory (important now that the strings are larger) Line 603: TestReadWriteStrings(PLAIN_ENCODING, "%066000d"); > I know I'm missing something obvious, but what makes this particular format It pads the string with up to 66000 zeros -- To view, visit http://gerrit.cloudera.org:8080/2725 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic27485bc0b2151e7c182c36e7ad6258dc549517b Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
