-
Notifications
You must be signed in to change notification settings - Fork 42
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] Remove dependency on json crate #2561
Conversation
a1ea9b7
to
4b7d6ca
Compare
@@ -157,6 +156,7 @@ public void postorder(DBSPTypeSemigroup type) { | |||
aggregates::*, | |||
variant::*, | |||
}; | |||
#[cfg(test)] |
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.
I see lines like this in the generated code:
sltsqlvalue::to_sql_row_impl! {
Tup16<T0, T1, T2, T3, T4, T5, T6, T7, T8, T9, T10, T11, T12, T13, T14, T15>,
Tup18<T0, T1, T2, T3, T4, T5, T6, T7, T8, T9, T10, T11, T12, T13, T14, T15, T16, T17>,
Tup11<T0, T1, T2, T3, T4, T5, T6, T7, T8, T9, T10>,
Tup12<T0, T1, T2, T3, T4, T5, T6, T7, T8, T9, T10, T11>,
Tup13<T0, T1, T2, T3, T4, T5, T6, T7, T8, T9, T10, T11, T12>,
Tup14<T0, T1, T2, T3, T4, T5, T6, T7, T8, T9, T10, T11, T12, T13>,
Tup15<T0, T1, T2, T3, T4, T5, T6, T7, T8, T9, T10, T11, T12, T13, T14>,
}
These are not gated by cfg(test)
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.
these should be gone now
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.
Fixes #2560
4500a56
to
7293372
Compare
7293372
to
32d659a
Compare
Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
This is a postfix for PR 2557, bumping the version of a transitive dependency on google-cloud-gax. Signed-off-by: Leonid Ryzhyk <leonid@feldera.com>
f279f24
to
13099ea
Compare
No description provided.