timsaucer commented on PR #711: URL: https://github.com/apache/datafusion-python/pull/711#issuecomment-2122437247
@Michael-J-Ward It looks like we have a _potential_ regression between 37.1.0 and 38.0.0. Namely `substr` on 37.1.0 would accept a start and length, the parameters that should apply to `substring`. I was using `substr` incorrectly but it worked by accident on 37.1.0 because `substr` and `substring` were calling the same underlying function. In 38.0.0 this is updated and `substr` now fails if you pass it two parameters. This PR includes exposing `substring` which was missing on the python side. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
