andygrove commented on PR #47: URL: https://github.com/apache/arrow-ballista/pull/47#issuecomment-1142299098
Having re-read the comments, I can see that this approach is not compatible with having a custom logical plan representation and this is not a feature I had considered. Also, I don't think we have any tests for this feature? @thinkharderdev Perhaps you could add a test for your use case? I will rethink the approach here. -- 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]
