-
Notifications
You must be signed in to change notification settings - Fork 312
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
feat: add DECIMAL and BIGDECIMAL as aliases for NUMERIC and BIGNUMERIC to scalar types enums #633
Comments
On second-thought, these are actually called 🤔 |
But these are defined as valid standard sql aliases here: https://cloud.google.com/bigquery/docs/reference/standard-sql/data-types#decimal_types |
IMO, this is a bug not a feature. :) |
@tswast shall I address this? |
Gotcha. I didn't realize the backend had these as aliases. Yes, let's go for it. |
I can't find any use of I searched:
Is there a reason we think we need this (other than for our internal use)? |
I'd like to use those in code samples where folks have STRUCT type query parameters with sub-fields Honestly, it'd be much more useful if we supplemented the class to allow a builder-style construction where we add a |
Closed by #638 |
I'm thinking that there's a lot to unpack there. :)
I'm lost. which class? Can you expand a little? |
I mean the ScalarQueryParameter type. I've filed #642 we can continue the discussion of the feature there.
Yes, I mean the
That's right! See: #463 In addition to what you've written with anonymous sub-fields, it's also possible to give those names. |
#626 (comment)
The text was updated successfully, but these errors were encountered: