huaxingao commented on PR #15175: URL: https://github.com/apache/iceberg/pull/15175#issuecomment-5516723452
@nastra Following up on my earlier question — I saw that the toString() route was attempted in #13251 and closed unmerged over @stevenzwu's point that it should only include the projected fields. Since this version doesn't touch api/ at all, I'll approve for now. Happy to file a follow-up issue for StructProjection.toString() if you'd like it tracked -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
