[Enhancement] modify statistic cache expire-update logical

[Seaven]

Why I'm doing:

What I'm doing:

Before this PR:

1. collect new statistics
2. drop old statistics cache
3. wait a moment and really update new statistics cache

This PR:

1. collect new statistics
2. update new statistics cache directly and replace old cache

Fixes #issue

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
• This is a backport pr

Bugfix cherry-pick branch check:

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

[starrocks-cr]

The most risky bug in this code is:
Potentially blocking operation in asynchronous execution.

You can modify the code like this:

try {
    ColumnBasicStatsCacheLoader loader = new ColumnBasicStatsCacheLoader();
    CompletableFuture<Map<ColumnStatsCacheKey, Optional<ColumnStatistic>>> future =
            loader.asyncLoadAll(cacheKeys, statsCacheRefresherExecutor);
    if (isSync) {
        // Use get() with a timeout to avoid indefinitely blocking if the operation hangs
        Map<ColumnStatsCacheKey, Optional<ColumnStatistic>> result = future.get(60, TimeUnit.SECONDS);
        columnStatistics.synchronous().putAll(result);
    } else {
        columnStatistics.synchronous().refresh();
        future.whenComplete((res, e) -> {
            if (e != null) {
                LOG.error("Error loading column statistics", e);
            } else {
                columnStatistics.synchronous().putAll(res);
            }
        });
    }
} catch (TimeoutException e) {
    LOG.error("Timeout occurred while refreshing column statistics", e);
} catch (Exception e) {
    LOG.warn("Failed to refresh getColumnStatistics", e);
}

This modification includes a timeout for the future.get() call to prevent indefinite blocking, and error handling within whenComplete to log any exceptions encountered during asynchronous execution.

[sonarcloud]

Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

[github-actions]

[Java-Extensions Incremental Coverage Report]

✅ pass : 0 / 0 (0%)

[github-actions]

[FE Incremental Coverage Report]

✅ pass : 51 / 57 (89.47%)

file detail

	path	covered_line	new_line	coverage	not_covered_line_detail
🔵	com/starrocks/sql/optimizer/statistics/StatisticStorage.java	0	1	00.00%	[97]
🔵	com/starrocks/sql/optimizer/statistics/CachedStatisticStorage.java	45	50	90.00%	[308, 309, 310, 313, 314]
🔵	com/starrocks/statistic/AnalyzeMgr.java	6	6	100.00%	[]

[github-actions]

[BE Incremental Coverage Report]

✅ pass : 0 / 0 (0%)