manisin commented on PR #6674: URL: https://github.com/apache/iceberg/pull/6674#issuecomment-1407070649
> the changes mostly LGTM, but it's unclear why we're removing tests with special characters. Thanks for the review @nastra ! The test with special characters are removed as I also removed the code for custom handling of the special characters. The special characters have be escaped/quoted correctly according to spark/snowflake documentation by the user and they would be passed through the catalog library to snowflake without any modification. -- 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]
