-
Notifications
You must be signed in to change notification settings - Fork 590
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
Support serial type (rowid) in sink #16009
Comments
Currently, we have supported int64 in protobuf format directly as a native number, but we haven't supported it in JSON and Avro formats. Because we need to use Avro doesn't have this issue, so we can use native int64. |
Agree that the Either plain string "12345" or hexadecimal "0x12345" looks good to me. |
One benefit of fixed-length If that is not a concern, or we already display |
I submitted a PR trying to use padded hexadecimal strings in the JSON. It looks ok.
|
One use case is that we can specify rowid in sink primary key, so that internal buffering in sink executor can be avoided for lower latency.
Similar to #15008, but we might need implicit cast.
The text was updated successfully, but these errors were encountered: