Skip to content

Conversation

@dramaticlly
Copy link
Contributor

@dramaticlly dramaticlly commented Jun 21, 2024

support of
SELECT *, _partition from iceberg.foo.bar when foo.bar table has over 1000 columns defined

@github-actions github-actions bot added the spark label Jun 21, 2024
@dramaticlly
Copy link
Contributor Author

Looks like PRB failed due to junit clean up of temp directory as mentioned in #10569

Copy link
Member

@szehon-ho szehon-ho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks @dramaticlly

@szehon-ho szehon-ho merged commit 6223708 into apache:main Jul 5, 2024
@szehon-ho
Copy link
Member

Merged, thanks @dramaticlly

@pan3793
Copy link
Member

pan3793 commented Jul 10, 2024

... when foo.bar table has over 1000 columns defined

@dramaticlly can you clarify why it was not supported before? where does the restriction come from? I don't see the magic number 1000 in the codebase

@dramaticlly
Copy link
Contributor Author

dramaticlly commented Jul 10, 2024

... when foo.bar table has over 1000 columns defined

@dramaticlly can you clarify why it was not supported before? where does the restriction come from? I don't see the magic number 1000 in the codebase

@pan3793 so it will only fix the scenario when selecting all fields together with partition metadata column on iceberg table with more than 1000 columns. The unit test shall reproduce the problem if fix is missing. As for the reasoning, the 1000 is coming from the default field id to be assigned for inner partition struct of iceberg table, more detailed analysis can be found in #9923

jasonf20 pushed a commit to jasonf20/iceberg that referenced this pull request Aug 4, 2024
zachdisc pushed a commit to zachdisc/iceberg that referenced this pull request Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants