Re: [PR] API: Speed up Timestamps#toHumanString [iceberg]

2025-03-11 Thread via GitHub
RussellSpitzer commented on PR #12447: URL: https://github.com/apache/iceberg/pull/12447#issuecomment-2715305062 Can you see who the caller is there? I'm just interested in why flink is calling that function so often -- This is an automated message from the Apache Git Service. To respond

Re: [PR] API: Speed up Timestamps#toHumanString [iceberg]

2025-03-11 Thread via GitHub
suneet-s commented on PR #12447: URL: https://github.com/apache/iceberg/pull/12447#issuecomment-2715260800 @RussellSpitzer Thanks for reviewing this change. In a performance test we were running where a flink pipieline was writing data to an iceberg table, we saw that this function was taki

Re: [PR] API: Speed up Timestamps#toHumanString [iceberg]

2025-03-11 Thread via GitHub
RussellSpitzer commented on code in PR #12447: URL: https://github.com/apache/iceberg/pull/12447#discussion_r1989662498 ## api/src/test/java/org/apache/iceberg/transforms/TestTransformUtil.java: ## @@ -0,0 +1,66 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API: Speed up Timestamps#toHumanString [iceberg]

2025-03-11 Thread via GitHub
RussellSpitzer commented on code in PR #12447: URL: https://github.com/apache/iceberg/pull/12447#discussion_r1989658472 ## api/src/test/java/org/apache/iceberg/transforms/TestTransformUtil.java: ## @@ -0,0 +1,66 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API: Speed up Timestamps#toHumanString [iceberg]

2025-03-11 Thread via GitHub
RussellSpitzer commented on code in PR #12447: URL: https://github.com/apache/iceberg/pull/12447#discussion_r1989657396 ## api/src/test/java/org/apache/iceberg/transforms/TestTransformUtil.java: ## @@ -0,0 +1,66 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under o

Re: [PR] API: Speed up Timestamps#toHumanString [iceberg]

2025-03-11 Thread via GitHub
RussellSpitzer commented on PR #12447: URL: https://github.com/apache/iceberg/pull/12447#issuecomment-2714915616 @suneet-s What is the motivation for this change? Is toHumanString on a critical path somewhere? I don't think I have an issue with putting in a faster implementation as long as

[PR] API: Speed up Timestamps#toHumanString [iceberg]

2025-03-03 Thread via GitHub
suneet-s opened a new pull request, #12447: URL: https://github.com/apache/iceberg/pull/12447 String#format uses regexes internally which can be expensive when used repeatedly. This PR switches out the implementation for a char array implementation that pads zeros as needed. A simple