Ankitraj-builds commented on issue #3977: URL: https://github.com/apache/iceberg-python/issues/3977#issuecomment-6322341586
Hi @ArulJerald, are you still working on this? I haven't seen a PR yet, so I wanted to check before starting. If it's free, I'd like to pick it up. My plan is the smaller fix: keep the existing Glue Parameters, but remove keys that were in the previous table properties and are not in the new ones. Glue keys that never came from Iceberg stay as they are. This matches what @David-Banquet found in the Java Glue catalog. I already have a test in tests/catalog/test_glue.py that sets two properties, removes one, commits, and checks it is gone from Glue. It fails on main and passes with the fix. @sungwy does this approach work for you, or would you prefer a full rebuild of Parameters? -- 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]
