Skip to content

Commit

Permalink
apacheGH-39413: [C++][Parquet] Vectorize decode plain on FLBA (apache…
Browse files Browse the repository at this point in the history
…#39414)

### Rationale for this change

### What changes are included in this PR?
FLBA Decode Plain is not vectorized. So this parsing can be implemented faster https://godbolt.org/z/xWeb93xjW

### Are these changes tested?
Yes, on unittest

### Are there any user-facing changes?

* Closes: apache#39413

Authored-by: Dmitry Stasenko <dmitry.stasenko@pinely.com>
Signed-off-by: mwish <maplewish117@gmail.com>
  • Loading branch information
Hattonuri authored Jan 2, 2024
1 parent 3087c94 commit 98f677a
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions cpp/src/parquet/encoding.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1080,9 +1080,7 @@ inline int DecodePlain<FixedLenByteArray>(const uint8_t* data, int64_t data_size
ParquetException::EofException();
}
for (int i = 0; i < num_values; ++i) {
out[i].ptr = data;
data += type_length;
data_size -= type_length;
out[i].ptr = data + i * type_length;
}
return static_cast<int>(bytes_to_decode);
}
Expand Down

0 comments on commit 98f677a

Please sign in to comment.