-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix query failure when accessing numeric types in BigQuery #10565
Conversation
329b3af
to
596a292
Compare
596a292
to
ebd83b9
Compare
@@ -190,7 +191,7 @@ else if (type.equals(TIME_MICROS)) { | |||
else if (javaType == double.class) { | |||
type.writeDouble(output, ((Number) value).doubleValue()); | |||
} | |||
else if (javaType == Slice.class) { | |||
else if (javaType == Slice.class || type.getJavaType() == Int128.class) { |
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.
writeSlice
method name is misleading.
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.
Oh - next commit :)
Can you add a regression test for this? |
@ksobolew The tests already exist in testBigNumericMapping and testNumericMapping in TestBigQueryTypeMapping. |
...and they must be run manually, unfortunately. Thanks! |
Fixes #10564
Verified all tests locally.