-
Notifications
You must be signed in to change notification settings - Fork 3.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[fix](Outfile) Fix the data type mapping for complex types in Doris to the ORC and Parquet file formats. #44041
base: master
Are you sure you want to change the base?
Conversation
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
run buildall |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
@@ -17,6 +17,7 @@ | |||
|
|||
#pragma once | |||
|
|||
#include <arrow/array/builder_binary.h> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
warning: 'arrow/array/builder_binary.h' file not found [clang-diagnostic-error]
#include <arrow/array/builder_binary.h>
^
TeamCity be ut coverage result: |
4d80aa1
to
6990720
Compare
3790342
to
c48f8dc
Compare
run buildall |
run buildall |
TeamCity be ut coverage result: |
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 39950 ms
|
TPC-DS: Total hot run time: 196086 ms
|
ClickBench: Total hot run time: 33.37 s
|
run buildall |
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 40632 ms
|
TPC-DS: Total hot run time: 195881 ms
|
ClickBench: Total hot run time: 33.54 s
|
@@ -48,28 +49,17 @@ Status DataTypeHLLSerDe::serialize_column_to_json(const IColumn& column, int64_t | |||
Status DataTypeHLLSerDe::serialize_one_cell_to_json(const IColumn& column, int64_t row_num, | |||
BufferWritable& bw, | |||
FormatOptions& options) const { | |||
if (!options._output_object_data) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only export use this option _output_object_data
?
What problem does this PR solve?
Problem Summary:
As before, the behavior of exporting of complex data types in Doris is as follows:
What's more, there are some issues when exporting complex data types to the ORC file format.
This PR does two things:
Release note
None
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)