yashmayya commented on PR #13212: URL: https://github.com/apache/pinot/pull/13212#issuecomment-2222951103
@Jackie-Jiang @xiangfu0 to resolve this impasse, I've also added support in this PR for handling transform function errors (largely taken from Xiang's PR - https://github.com/apache/pinot/pull/12763). We can add @xiangfu0 as a co-author ([ref](https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors#creating-co-authored-commits-on-github)) if / when this PR is merged. One minor difference here is that I'm only handling the case where the invocation itself results in an error, and not when there is an unsupported output value class which seems more like a hard error. Also, I've written some additional tests. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
