Skip to content

Commit b6b1947

Browse files
Merge pull request #10964 from ObjatieGroba/bugfix_raw_column_size
Fix incorrect Column byte size
2 parents caab379 + ce451bf commit b6b1947

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

src/Columns/ColumnDecimal.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ class ColumnDecimal final : public COWHelper<ColumnVectorHelper, ColumnDecimal<T
113113

114114
Field operator[](size_t n) const override { return DecimalField(data[n], scale); }
115115

116-
StringRef getRawData() const override { return StringRef(reinterpret_cast<const char*>(data.data()), data.size()); }
116+
StringRef getRawData() const override { return StringRef(reinterpret_cast<const char*>(data.data()), byteSize()); }
117117
StringRef getDataAt(size_t n) const override { return StringRef(reinterpret_cast<const char *>(&data[n]), sizeof(data[n])); }
118118
void get(size_t n, Field & res) const override { res = (*this)[n]; }
119119
bool getBool(size_t n) const override { return bool(data[n]); }

src/Columns/ColumnVector.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ class ColumnVector final : public COWHelper<ColumnVectorHelper, ColumnVector<T>>
264264

265265
bool isFixedAndContiguous() const override { return true; }
266266
size_t sizeOfValueIfFixed() const override { return sizeof(T); }
267-
StringRef getRawData() const override { return StringRef(reinterpret_cast<const char*>(data.data()), data.size()); }
267+
StringRef getRawData() const override { return StringRef(reinterpret_cast<const char*>(data.data()), byteSize()); }
268268

269269

270270
bool structureEquals(const IColumn & rhs) const override

0 commit comments

Comments
 (0)