feat(common): introduce Serial as Scalar and ScalarRef #8327
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
This pull request adds a new module,
serial_array
, to thesrc/common/src/array/
directory, which contains an implementation for theSerial
type.Serial
has been included in the list ofScalar
variants insrc/common/src/types/mod.rs
, with corresponding implementations forScalar
andScalarRef
. TheScalarRefImpl
's implementation ofToBinary
now includesSerial
in the match statement. Additionally,serialize_scalar()
insrc/common/src/util/value_encoding/mod.rs
has been modified to handleSerial
.Checklist For Contributors
- [ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features Sqlsmith: Sql feature generation #7934)../risedev check
(or alias,./risedev c
)Checklist For Reviewers
Documentation
Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.
Release note