Skip to content

Commit

Permalink
Fix delta filter deserialization for format ver 19. (#4541)
Browse files Browse the repository at this point in the history
This fixes the delta filter deserialization for version 19. In that
version, we already included the extra reinterpret type.

---
TYPE: BUG
DESC: Fix delta filter deserialization for format ver 19.
  • Loading branch information
KiterLuc authored Nov 27, 2023
1 parent 217be37 commit 2b1ada7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions tiledb/sm/filter/filter_create.cc
Original file line number Diff line number Diff line change
Expand Up @@ -126,8 +126,8 @@ shared_ptr<tiledb::sm::Filter> tiledb::sm::FilterCreate::deserialize(
uint8_t compressor_char = deserializer.read<uint8_t>();
int compression_level = deserializer.read<int32_t>();
Datatype reinterpret_type = Datatype::ANY;
if (version >= 20 && (filtertype == FilterType::FILTER_DELTA ||
filtertype == FilterType::FILTER_DOUBLE_DELTA)) {
if ((version >= 20 && filtertype == FilterType::FILTER_DOUBLE_DELTA) ||
(version >= 19 && filtertype == FilterType::FILTER_DELTA)) {
uint8_t reinterpret = deserializer.read<uint8_t>();
reinterpret_type = static_cast<Datatype>(reinterpret);
}
Expand Down

0 comments on commit 2b1ada7

Please sign in to comment.