[RocksDB] [Performance Branch] Revert previous patch.

haoboxu · haoboxu · commit bf4a48ccb356 · 2013-12-20T18:20:06.000-08:00
Summary: The previous patch is wrong. rep_.resize(kHeader) just resets the header portion to zero, and should not cause a re-allocation if g++ does it right. I will go ahead and revert it. Test Plan: make check Reviewers: dhruba, sdong CC: leveldb Differential Revision: https://reviews.facebook.net/D14793
diff --git a/db/write_batch.cc b/db/write_batch.cc
@@ -58,6 +58,7 @@ bool WriteBatch::Handler::Continue() {
 
 void WriteBatch::Clear() {
   rep_.clear();
+  rep_.resize(kHeader);
 }
 
 int WriteBatch::Count() const {

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,7 @@ bool WriteBatch::Handler::Continue() {`
`58`	`58`
`59`	`59`	`void WriteBatch::Clear() {`
`60`	`60`	`rep_.clear();`
	`61`	`+ rep_.resize(kHeader);`
`61`	`62`	`}`
`62`	`63`
`63`	`64`	`int WriteBatch::Count() const {`