Skip to content

[common] Replace commons StringUtils usages with Paimon StringUtils#7603

Open
leaves12138 wants to merge 1 commit intoapache:masterfrom
leaves12138:ban-commons-stringutils
Open

[common] Replace commons StringUtils usages with Paimon StringUtils#7603
leaves12138 wants to merge 1 commit intoapache:masterfrom
leaves12138:ban-commons-stringutils

Conversation

@leaves12138
Copy link
Copy Markdown
Contributor

Summary

  • replace shaded and commons-lang3 StringUtils usages with org.apache.paimon.utils.StringUtils
  • add the minimal missing helper methods required by the current call sites
  • add compatibility tests for the copied helpers and forbid commons StringUtils imports in checkstyle

Verification

  • grep -R -nE "org\.apache\.paimon\.shade\.org\.apache\.commons\.lang3\.StringUtils|org\.apache\.commons\.lang3\.StringUtils" . --exclude-dir=.git
  • git diff --check
  • mvn -pl paimon-common -am -Dspotless.check.skip -Denforcer.skip -Dmaven.test.skip=true install
  • mvn -pl paimon-bundle -am -Dspotless.check.skip -Denforcer.skip -Dmaven.test.skip=true install
  • mvn -pl paimon-flink/paimon-flink-common -am -Dspotless.check.skip -Denforcer.skip -Dmaven.test.skip=true install
  • mvn -pl paimon-flink/paimon-flink-1.18 -am -rf :paimon-flink-1.18 -Dspotless.check.skip -Denforcer.skip -Dmaven.test.skip=true compile

@leaves12138 leaves12138 changed the title Replace commons StringUtils usages with Paimon StringUtils [WIP] Replace commons StringUtils usages with Paimon StringUtils Apr 7, 2026
@leaves12138 leaves12138 force-pushed the ban-commons-stringutils branch 2 times, most recently from 0798c5a to 0beaad9 Compare April 10, 2026 06:47
@leaves12138 leaves12138 changed the title [WIP] Replace commons StringUtils usages with Paimon StringUtils [common] Replace commons StringUtils usages with Paimon StringUtils Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant