Ville Brofeldt
Ville Brofeldt
Thanks @amitmiran137 and @srinisubramanian ! so we're ok closing this with the assumption that upgrading to 1.5 should fix the problem?
FYI I believe I've hit this issue, and I'm working on a fix for it
@bryanjtc thanks for the reminder. This has been open long enough, so I started CI flow and can merge once it's ready if CI is green
> Could you add some unit tests to make sure that the commands are working as expected. FYI @AAfghahi I removed the change request from your review as the requested...
@bkyryliuk it seems there has been efforts to get Spark SQL in for a long time, but many previous PRs have gone stale in the end. As potential problems are...
@bkyryliuk I can help write some tests if necessary, I think this would be a really nice feature to have.
This PR seems to be stale (is in a conflicted state + review comment hasn't been responded to), but there is another one that should cover this use case: #23344...
Closing as a rewrite of this PR has already been merged.
@cccs-RyanK I believe this needs a rebase
@aehanno can you show a before and after of these changes? I'd like to understand how these translations are being picked up