-
Notifications
You must be signed in to change notification settings - Fork 28k
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
[SPARK-41554] fix changing of Decimal scale when scale decreased by m… #39381
Conversation
Can one of the admins verify this patch? |
Is there a backport for 3.3. too? |
Created #39813. Sorry for the delay. |
Roger that - any chance you can retrigger these tests? they're hung for some reason |
I'll merge these if we can get tests to re-run, try retriggering them |
I tried retriggering with a new push above, but it didn't help, seems like the tests passed, but it was still 'Queued - Test results' status on this page. I've just retriggered them again, but now from the github UI. Let's see. |
I'm confused, it looks like it succeeded if I click through to the results. Maybe it has succeeded? I guess I can assume so |
I merged to 3.2. The relevant test(s) has/have actually passed |
We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable. |
…ore than 18 This is a backport PR for apache#39099 Closes apache#39381 from fe2s/branch-3.2-fix-decimal-scaling. Authored-by: oleksii.diagiliev <oleksii.diagiliev@workday.com> Signed-off-by: Sean Owen <srowen@gmail.com>
…ore than 18
This is a backport PR for #39099