findepi commented on issue #9276: URL: https://github.com/apache/iceberg/issues/9276#issuecomment-2175479755
> Does Iceberg support using its libraries in user-defined locales? Not for me to decide this, but i believe that we have basically these options 1. make the code independent of JVM locale (Palantir's error-prone check [DefaultLocale](https://github.com/palantir/gradle-baseline/blob/develop/baseline-error-prone/src/main/java/com/palantir/baseline/errorprone/DefaultLocale.java) can be helpful) 2. test the code on various locales including Turkish 3. add a runtime check enforcing JVM locale Of these I think (3) is least awesome, because Iceberg is a library and should support being embedded in various contesxts. I think we should do (1). -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
