Skip to content

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

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

[WIP] 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
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