-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update to arrow 38 #6115
Update to arrow 38 #6115
Conversation
@@ -1089,7 +1089,7 @@ mod tests { | |||
#[tokio::test] | |||
async fn test_sort_fetch_memory_calculation() -> Result<()> { | |||
// This test mirrors down the size from the example above. | |||
let avg_batch_size = 6000; | |||
let avg_batch_size = 5000; |
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.
Arrays are smaller because of apache/arrow-rs#4061
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.
Looks good to me -- thank you @tustvold
🤔 there appear to be more changes needed to tests in other modules |
@@ -303,7 +303,7 @@ select arrow_cast('30 minutes', 'Interval(MonthDayNano)'); | |||
|
|||
## Duration | |||
|
|||
query error DataFusion error: Error during planning: Cannot automatically convert Interval\(MonthDayNano\) to Duration\(Second\) | |||
query error DataFusion error: This feature is not implemented: Can't create a scalar from array of type "Duration\(Second\)" |
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 think this is the result of apache/arrow-rs#4020
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.
LGTM -- thank you @tustvold
Which issue does this PR close?
Closes #.
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?