-
Notifications
You must be signed in to change notification settings - Fork 35
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
[SQL] Add support for 'default' column values #1061
Conversation
@mihaibudiu , do we need to impose any restrictions on default values to make sure they are deterministic, so that replaying the same Kafka messages results in the same values in input tables? |
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
Optimize `deserialize_table_record` and `deserialize_struct` macros. Instead of assigning default values to fields before deserializing the struct, we deserialize first and only assign defaults to missing fields. Signed-off-by: Leonid Ryzhyk <leonid@feldera.com>
Right now we don't have support for any non deterministic function |
* [SQL] Add parser support for 'default' column values * [SQL] Emit serde code for default values * Optimize the handling of defaults in deserialization. Optimize `deserialize_table_record` and `deserialize_struct` macros. Instead of assigning default values to fields before deserializing the struct, we deserialize first and only assign defaults to missing fields. Signed-off-by: Mihai Budiu <mbudiu@feldera.com> Co-authored-by: Leonid Ryzhyk <leonid@feldera.com>
Is this a user-visible change (yes/no): yes
Fixes #1058