Merge pull request #2 from xuepanchen/xuepanchen-arrow-1712

xuepanchen · web-flow · commit d021c54bb393 · 2018-01-15T14:22:04.000-05:00
Update BinaryBuilder::Resize(int64_t capacity) in builder.cc
diff --git a/cpp/src/arrow/builder.cc b/cpp/src/arrow/builder.cc
@@ -1223,6 +1223,7 @@ Status BinaryBuilder::Resize(int64_t capacity) {
   DCHECK_LT(capacity, std::numeric_limits<int32_t>::max());
   // one more then requested for offsets
   RETURN_NOT_OK(offsets_builder_.Resize((capacity + 1) * sizeof(int64_t)));
+  RETURN_NOT_OK(value_data_builder_.Resize(capacity));
   return ArrayBuilder::Resize(capacity);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -1223,6 +1223,7 @@ Status BinaryBuilder::Resize(int64_t capacity) {`
`1223`	`1223`	`DCHECK_LT(capacity, std::numeric_limits<int32_t>::max());`
`1224`	`1224`	`// one more then requested for offsets`
`1225`	`1225`	`RETURN_NOT_OK(offsets_builder_.Resize((capacity + 1) * sizeof(int64_t)));`
	`1226`	`+ RETURN_NOT_OK(value_data_builder_.Resize(capacity));`
`1226`	`1227`	`return ArrayBuilder::Resize(capacity);`
`1227`	`1228`	`}`
`1228`	`1229`