-
Notifications
You must be signed in to change notification settings - Fork 843
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
Replace the fn get_data_type
by const DATA_TYPE
in BinaryArray and StringArray
#2289
Replace the fn get_data_type
by const DATA_TYPE
in BinaryArray and StringArray
#2289
Conversation
Signed-off-by: remzi <[email protected]>
Signed-off-by: remzi <[email protected]>
BinaryArray::get_data_type
and StringArray::get_data_type
as const
functions BinaryArray::get_data_type
and StringArray::get_data_type
as const
values
Please add the |
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.
Perhaps we could first deprecate get_data_type() to avoid this being a breaking change?
Signed-off-by: remzi <[email protected]>
BinaryArray::get_data_type
and StringArray::get_data_type
as const
valuesfn get_data_type
by const DATA_TYPE
in BinaryArray and StringArray
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.
This is a very nice improvement @HaoYang670 -- thank you
} else { | ||
DataType::Binary | ||
} | ||
#[deprecated(note = "please use `Self::DATA_TYPE` instead")] |
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.
💯 for deprecation
Benchmark runs are scheduled for baseline = 299908e and contender = 1cc8563. 1cc8563 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Signed-off-by: remzi [email protected]
Which issue does this PR close?
Related to #2274.
Rationale for this change
More constant evaluation is better.
What changes are included in this PR?
get_data_type
functionconst DATA_TYPE
valueAre there any user-facing changes?
Yes.