[BugFix] fix concurrency modify propertyInfo in empty PhysicalPropertySet

[packy92]

Fixes #issue
Fix stack like this

java.util.ConcurrentModificationException: null
        at java.util.HashMap.computeIfAbsent(HashMap.java:1134) ~[?:?]
        at com.starrocks.sql.optimizer.base.DistributionDisjointSet.find(DistributionDisjointSet.java:55) ~[starrocks-fe.jar:?]
        at com.starrocks.sql.optimizer.base.DistributionDisjointSet.union(DistributionDisjointSet.java:74) ~[starrocks-fe.jar:?]
        at com.starrocks.sql.optimizer.base.DistributionSpec$PropertyInfo.unionDistributionCols(DistributionSpec.java:93) ~[starrocks-fe.jar:?]
        

The root cause is

private PhysicalPropertySet computeShuffleJoinOutputProperty(JoinOperator joinType,
                                                                 List<DistributionCol> leftShuffleColumns,
                                                                 List<DistributionCol> rightShuffleColumns) {
        Optional<HashDistributionDesc> requiredShuffleDesc = getRequiredShuffleDesc();
        if (!requiredShuffleDesc.isPresent()) {
            return PhysicalPropertySet.EMPTY;
        }
}

may return PhysicalPropertySet.EMPTY and we may update the PropertyInfo in DistributionProperty.EMPTY in the next computeHashJoinDistributionPropertyInfo() step which may concurrent modify a static object at the same time.

What type of PR is this:

• BugFix
• Feature
• Enhancement
• Refactor
• UT
• Doc
• Tool

Does this PR entail a change in behavior?

• Yes, this PR will result in a change in behavior.
• No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

• Interface/UI changes: syntax, type conversion, expression evaluation, display information
• Parameter changes: default values, similar parameters but with different default values
• Policy changes: use new policy to replace old one, functionality automatically enabled
• Feature removed
• Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

• I have added test cases for my bug fix or my new feature
• This pr needs user documentation (for new or modified features or behaviors)
  • I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

• I have checked the version labels which the pr will be auto-backported to the target branch
  • 3.2
  • 3.1
  • 3.0
  • 2.5

[github-actions]

[FE Incremental Coverage Report]

✅ pass : 2 / 2 (100.00%)

file detail

	path	covered_line	new_line	coverage	not_covered_line_detail
🔵	com/starrocks/sql/optimizer/OutputPropertyDeriver.java	2	2	100.00%	[]

[github-actions]

[BE Incremental Coverage Report]

✅ pass : 0 / 0 (0%)

[wanpengfei-git]

@Mergifyio backport branch-3.1

[wanpengfei-git]

@Mergifyio backport branch-3.0

[mergify]

backport branch-3.1

✅ Backports have been created

• #34146 [BugFix] fix concurrency modify propertyInfo in empty PhysicalPropertySet (backport #34140) has been created for branch branch-3.1

[wanpengfei-git]

@Mergifyio backport branch-2.5

[mergify]

backport branch-3.0

✅ Backports have been created

• #34147 [BugFix] fix concurrency modify propertyInfo in empty PhysicalPropertySet (backport #34140) has been created for branch branch-3.0 but encountered conflicts

[mergify]

backport branch-2.5

✅ Backports have been created

• #34148 [BugFix] fix concurrency modify propertyInfo in empty PhysicalPropertySet (backport #34140) has been created for branch branch-2.5 but encountered conflicts