Uh oh!
There was an error while loading. Please reload this page.
API: Override StructProjection toString with that of underlying struct - #13251
API: Override StructProjection toString with that of underlying struct#13251manuzhang wants to merge 1 commit into
Conversation
szehon-ho
commented
Jun 10, 2025
hm not opposed to change, but curious, would it be better to also print the projection? Do you have a particular projection in mind you want toString for? |
manuzhang
commented
Jun 17, 2025
@szehon-ho In the case of #12414, it's |
| @Override | ||
| public String toString() { | ||
| return struct.toString(); |
There was a problem hiding this comment.
I would disagree that StructProjection#toString print the full wrapped struct. It should only print the projected fields.
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions. |
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
Closes#12414