mihaibudiu opened a new pull request, #3708: URL: https://github.com/apache/calcite/pull/3708
This is the second attempt to provide a partial fix for [CALCITE-6248]. The fix is partial, because the bug is in Avatica, but Avatica CI fails if this fix is not applied. I have already merged a PR for this issue, but here I revert the changes in that prior PR. I apologize for this change of mind, it has become apparent that the previous choice for the fix was incorrect. The fix is about CAST(string to DATE). The assumption is that this cast should be more lenient than DATE literals in parsing the string, but should still require the date to be correct. The plan is to merge this PR, then attempt to merge the corresponding Avatica PR: https://github.com/apache/calcite-avatica/pull/238. If that PR passes, it can be merged. Then some leftover code in Calcite can be cleaned-up when a new Calcite release picks up a new Avatica release. Unfortunately I don't know of an easy way to test this PR before it's merged. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
