You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "BiteTheDDDDt (via GitHub)" <gi...@apache.org> on 2026/02/05 05:35:19 UTC
[PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
BiteTheDDDDt opened a new pull request, #60526:
URL: https://github.com/apache/doris/pull/60526
### What problem does this PR solve?
This pull request introduces a new serialization format for `DataTypeFixedLengthObject` columns, leveraging streamvbyte encoding for efficient storage and transmission of large data blocks. The new format is activated for BE exec version 10 and above, which is now set as the maximum supported version. Additionally, the `AggregateFunctionCount` and `AggregateFunctionCountNotNullUnary` functions are marked as trivial, likely for optimization purposes. Below are the most important changes:
### Serialization and Deserialization Improvements
* Introduced a new serialization/deserialization format for `DataTypeFixedLengthObject` that uses streamvbyte encoding for large data, improving efficiency for big data columns. The new logic is gated behind BE exec version 10 and includes fallback to the previous format for older versions (`be/src/vec/data_types/data_type_fixed_length_object.cpp`). [[1]](diffhunk://#diff-7d29ab3e43d23db58f2216e23cc131705067e133fb7ab2da72f2e67c725beb48L36-L41) [[2]](diffhunk://#diff-7d29ab3e43d23db58f2216e23cc131705067e133fb7ab2da72f2e67c725beb48L56-R130) [[3]](diffhunk://#diff-7d29ab3e43d23db58f2216e23cc131705067e133fb7ab2da72f2e67c725beb48L84-R144) [[4]](diffhunk://#diff-7d29ab3e43d23db58f2216e23cc131705067e133fb7ab2da72f2e67c725beb48R156-R172)
* Updated the calculation of uncompressed serialized bytes to account for the new serialization format and potential streamvbyte compression (`be/src/vec/data_types/data_type_fixed_length_object.cpp`).
* Added the `streamvbyte` library include to support the new encoding/decoding logic (`be/src/vec/data_types/data_type_fixed_length_object.cpp`).
### Version Management
* Increased `BeExecVersionManager::max_be_exec_version` from 8 to 10, with detailed documentation and warnings about the sensitivity of this field. The new version enables the updated serialization logic (`be/src/agent/be_exec_version_manager.cpp`).
* Defined a new constant `USE_NEW_FIXED_OBJECT_SERIALIZATION_VERSION = 10` to clearly mark the threshold for the new serialization format (`be/src/agent/be_exec_version_manager.h`).
### Aggregate Function Optimization
* Marked `AggregateFunctionCount` and `AggregateFunctionCountNotNullUnary` as trivial by overriding the `is_trivial()` method to return `true`, which may allow for performance optimizations in the aggregation engine (`be/src/vec/aggregate_functions/aggregate_function_count.h`). [[1]](diffhunk://#diff-a5dbb09237f197bffdcbd3bec4fdd089913ec143d96806618c8eeb4c5dbb8cfeR64-R65) [[2]](diffhunk://#diff-a5dbb09237f197bffdcbd3bec4fdd089913ec143d96806618c8eeb4c5dbb8cfeR212-R213)
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [ ] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [ ] No.
- [ ] Yes. <!-- Add document PR link here. eg: https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should merge into -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3859306368
# BE Regression && UT Coverage Report
Increment line coverage `96.61% (57/59)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60526_430d5297faaa6070e97018f2e5caa758e54ec7f1_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60526_430d5297faaa6070e97018f2e5caa758e54ec7f1_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.63% (25900/36157) |
| Line Coverage | 54.27% (270722/498804) |
| Region Coverage | 51.63% (224950/435687) |
| Branch Coverage | 53.14% (96610/181817) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "Thearas (via GitHub)" <gi...@apache.org>.
Thearas commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3851189025
Thank you for your contribution to Apache Doris.
Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+your+PR).
Please clearly describe your PR:
1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
3. What features were added. Why was this function added?
4. Which code was refactored and why was this part of the code refactored?
5. Which functions were optimized and what is the difference before and after the optimization?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "BiteTheDDDDt (via GitHub)" <gi...@apache.org>.
BiteTheDDDDt commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3852075951
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3858809881
# BE UT Coverage Report
Increment line coverage `94.92% (56/59)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/430d5297faaa6070e97018f2e5caa758e54ec7f1_430d5297faaa6070e97018f2e5caa758e54ec7f1/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/430d5297faaa6070e97018f2e5caa758e54ec7f1_430d5297faaa6070e97018f2e5caa758e54ec7f1/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 52.60% (19409/36896) |
| Line Coverage | 36.12% (180588/500028) |
| Region Coverage | 32.44% (139904/431303) |
| Branch Coverage | 33.46% (60607/181109) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "BiteTheDDDDt (via GitHub)" <gi...@apache.org>.
BiteTheDDDDt commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3851193945
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3858269582
# FE UT Coverage Report
Increment line coverage `` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60526_430d5297faaa6070e97018f2e5caa758e54ec7f1/fe_increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60526_430d5297faaa6070e97018f2e5caa758e54ec7f1/fe_report/index.html)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "zclllyybb (via GitHub)" <gi...@apache.org>.
zclllyybb merged PR #60526:
URL: https://github.com/apache/doris/pull/60526
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3851389500
# FE UT Coverage Report
Increment line coverage `` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60526_21cc719bbfbd17ffe3481317bd00f00173e5b1e3/fe_increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60526_21cc719bbfbd17ffe3481317bd00f00173e5b1e3/fe_report/index.html)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "Copilot (via GitHub)" <gi...@apache.org>.
Copilot commented on code in PR #60526:
URL: https://github.com/apache/doris/pull/60526#discussion_r2767237937
##########
be/src/vec/data_types/data_type_fixed_length_object.cpp:
##########
@@ -53,26 +85,49 @@ char* DataTypeFixedLengthObject::serialize(const IColumn& column, char* buf,
DCHECK(src_col.item_size() > 0)
<< "[serialize]item size of DataTypeFixedLengthObject should be greater than 0";
- // item size
unaligned_store<size_t>(buf, src_col.item_size());
buf += sizeof(size_t);
- // column data
+
const auto* origin_data = src_col.get_data().data();
memcpy(buf, origin_data, real_need_copy_num * src_col.item_size());
buf += real_need_copy_num * src_col.item_size();
-
return buf;
}
const char* DataTypeFixedLengthObject::deserialize(const char* buf, MutableColumnPtr* column,
int be_exec_version) const {
- //const flag
+ if (be_exec_version >= USE_NEW_FIXED_OBJECT_SERIALIZATION_VERSION) {
+ // New deserialization with streamvbyte decoding for large data
+ size_t real_have_saved_num = 0;
+ buf = deserialize_const_flag_and_row_num(buf, column, &real_have_saved_num);
+
+ auto& dst_col = assert_cast<ColumnType&>(*(column->get()));
Review Comment:
The original column pointer must be saved before calling deserialize_const_flag_and_row_num, similar to how it's done in DataTypeDecimal at line 178. The deserialize_const_flag_and_row_num function may wrap the column in a ColumnConst, and subsequent operations need to work on the underlying data column, not the wrapper. The correct pattern is:
1. Save the original column pointer: auto* origin_column = column->get();
2. Call deserialize_const_flag_and_row_num
3. Use origin_column for subsequent operations
```suggestion
auto* origin_column = column->get();
size_t real_have_saved_num = 0;
buf = deserialize_const_flag_and_row_num(buf, column, &real_have_saved_num);
auto& dst_col = assert_cast<ColumnType&>(*origin_column);
```
##########
be/src/vec/data_types/data_type_fixed_length_object.cpp:
##########
@@ -97,6 +153,23 @@ const char* DataTypeFixedLengthObject::deserialize(const char* buf, MutableColum
// data : item data1 | item data2...
int64_t DataTypeFixedLengthObject::get_uncompressed_serialized_bytes(const IColumn& column,
int be_exec_version) const {
+ if (be_exec_version >= USE_NEW_FIXED_OBJECT_SERIALIZATION_VERSION) {
+ // New format size calculation with streamvbyte
+ auto size = sizeof(bool) + sizeof(size_t) + sizeof(size_t) + sizeof(size_t);
+ auto real_need_copy_num = is_column_const(column) ? 1 : column.size();
+ const auto& src_col = assert_cast<const ColumnType&>(column);
Review Comment:
When the column is a ColumnConst, directly casting it to ColumnType will fail. The code should first unwrap the const column to get the underlying data column before casting, similar to how the old format handles it at lines 175-178. The serialize function correctly handles this at line 41 using serialize_const_flag_and_row_num which unwraps const columns, but get_uncompressed_serialized_bytes does not perform this unwrapping.
```suggestion
const IColumn* data_column = &column;
auto real_need_copy_num = column.size();
if (is_column_const(column)) {
const auto& const_column = assert_cast<const ColumnConst&>(column);
data_column = &(const_column.get_data_column());
real_need_copy_num = 1;
}
const auto& src_col = assert_cast<const ColumnType&>(*data_column);
```
##########
be/src/vec/data_types/data_type_fixed_length_object.cpp:
##########
@@ -97,6 +153,23 @@ const char* DataTypeFixedLengthObject::deserialize(const char* buf, MutableColum
// data : item data1 | item data2...
int64_t DataTypeFixedLengthObject::get_uncompressed_serialized_bytes(const IColumn& column,
int be_exec_version) const {
+ if (be_exec_version >= USE_NEW_FIXED_OBJECT_SERIALIZATION_VERSION) {
+ // New format size calculation with streamvbyte
+ auto size = sizeof(bool) + sizeof(size_t) + sizeof(size_t) + sizeof(size_t);
+ auto real_need_copy_num = is_column_const(column) ? 1 : column.size();
+ const auto& src_col = assert_cast<const ColumnType&>(column);
+ auto mem_size = src_col.item_size() * real_need_copy_num;
+ if (mem_size <= SERIALIZED_MEM_SIZE_LIMIT) {
+ return size + mem_size;
+ } else {
+ // Throw exception if mem_size is large than UINT32_MAX
Review Comment:
The comment has a grammatical error: "large than" should be "larger than".
##########
be/src/vec/data_types/data_type_fixed_length_object.cpp:
##########
@@ -33,12 +34,43 @@ namespace doris::vectorized {
char* DataTypeFixedLengthObject::serialize(const IColumn& column, char* buf,
int be_exec_version) const {
- // const flag
+ if (be_exec_version >= USE_NEW_FIXED_OBJECT_SERIALIZATION_VERSION) {
+ // New serialization with streamvbyte encoding for large data
+ const auto* data_column = &column;
+ size_t real_need_copy_num = 0;
+ buf = serialize_const_flag_and_row_num(&data_column, buf, &real_need_copy_num);
+
+ const auto& src_col = assert_cast<const ColumnType&>(*data_column);
+ DCHECK(src_col.item_size() > 0)
+ << "[serialize]item size of DataTypeFixedLengthObject should be greater than 0";
+
+ // item size
+ unaligned_store<size_t>(buf, src_col.item_size());
+ buf += sizeof(size_t);
+
+ auto mem_size = real_need_copy_num * src_col.item_size();
+ const auto* origin_data = src_col.get_data().data();
+
+ // column data
+ if (mem_size <= SERIALIZED_MEM_SIZE_LIMIT) {
+ memcpy(buf, origin_data, mem_size);
+ return buf + mem_size;
+ } else {
+ // Throw exception if mem_size is large than UINT32_MAX
Review Comment:
The comment has a grammatical error: "large than" should be "larger than". Additionally, the comment is misleading because streamvbyte_encode will not throw an exception if mem_size is larger than UINT32_MAX; instead, cast_set will throw an exception during the conversion at line 61. Consider revising the comment to accurately reflect this behavior.
```suggestion
// cast_set will throw an exception if mem_size exceeds UINT32_MAX during conversion
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3852502105
# FE UT Coverage Report
Increment line coverage `` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60526_8224ee7a1e1263ecf59bff98c091d69b9c3e1b45/fe_increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60526_8224ee7a1e1263ecf59bff98c091d69b9c3e1b45/fe_report/index.html)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "BiteTheDDDDt (via GitHub)" <gi...@apache.org>.
BiteTheDDDDt commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3851196710
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3869671737
PR approved by at least one committer and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "hello-stephen (via GitHub)" <gi...@apache.org>.
hello-stephen commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3860002769
# BE Regression && UT Coverage Report
Increment line coverage `96.61% (57/59)` :tada:
[Increment coverage report](http://coverage.selectdb-in.cc/coverage/60526_430d5297faaa6070e97018f2e5caa758e54ec7f1_merge/increment_report/index.html)
[Complete coverage report](http://coverage.selectdb-in.cc/coverage/60526_430d5297faaa6070e97018f2e5caa758e54ec7f1_merge/report/index.html)
| Category | Coverage |
|-------------------|--------------------|
| Function Coverage | 71.64% (25902/36157) |
| Line Coverage | 54.28% (270766/498804) |
| Region Coverage | 51.67% (225106/435687) |
| Branch Coverage | 53.15% (96643/181817) |
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3869671852
PR approved by anyone and no changes requested.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org
Re: [PR] [Improvement](serialize) use streamvbyte_encode in DataTypeFixedLengthObject::serialize [doris]
Posted by "BiteTheDDDDt (via GitHub)" <gi...@apache.org>.
BiteTheDDDDt commented on PR #60526:
URL: https://github.com/apache/doris/pull/60526#issuecomment-3858051637
run buildall
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org