Skip to content

[HotFix] Class name for logger in RemoteLogIndexCache#1265

Merged
luoyuxia merged 1 commit intoapache:mainfrom
polyzos:remote-index-cache-log-fix
Jul 4, 2025
Merged

[HotFix] Class name for logger in RemoteLogIndexCache#1265
luoyuxia merged 1 commit intoapache:mainfrom
polyzos:remote-index-cache-log-fix

Conversation

@polyzos
Copy link
Copy Markdown
Contributor

@polyzos polyzos commented Jul 4, 2025

No description provided.

@polyzos polyzos requested a review from Copilot July 4, 2025 05:49
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This hotfix corrects the logger instantiation in RemoteLogIndexCache to use its own class name for clearer logging.

  • Updated LoggerFactory.getLogger to reference RemoteLogIndexCache.class instead of RemoteLogManager.class.
Comments suppressed due to low confidence (1)

fluss-server/src/main/java/com/alibaba/fluss/server/log/remote/RemoteLogIndexCache.java:94

  • Great catch—using RemoteLogIndexCache.class ensures log entries correctly identify this class instead of RemoteLogManager.
    private static final Logger LOG = LoggerFactory.getLogger(RemoteLogIndexCache.class);

@polyzos polyzos requested a review from luoyuxia July 4, 2025 06:34
Copy link
Copy Markdown
Contributor

@luoyuxia luoyuxia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! LGTM!

@luoyuxia luoyuxia merged commit 56062d8 into apache:main Jul 4, 2025
4 of 5 checks passed
drr00t pushed a commit to drr00t/fluss that referenced this pull request Jul 5, 2025
polyzos added a commit to polyzos/fluss that referenced this pull request Aug 30, 2025
polyzos added a commit to Alibaba-HZY/fluss that referenced this pull request Aug 31, 2025
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.

3 participants