singhpk234 commented on PR #6576: URL: https://github.com/apache/iceberg/pull/6576#issuecomment-1382673430
did some more digging, posting what i found so far, this issue, is only observed in 0.14.x and 1.0.0 (and I directly tested my fix on top of master :sweat_smile:), 1.1.0 & master are fine, This is because starting 1.1.0 we now create a glue table (dummy table if the commit not succeeds, to be removed later) before writing s3 metadata files (which does the isTableEnroledinLF check, and now since the table is already created this check passes), prior to this the table creation use to happen after s3 write, which would result in failure. This above behaviour makes sense considering that this pr should no longer be required atleast to the master (happy to create backport pr's for lower versions if required), hence closing this pr. -- 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]
