Skip to content

[HotFix] Class name for logger in RemoteLogIndexCache#1265

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

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

Conversation

@polyzos

@polyzos polyzos commented Jul 4, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@polyzos polyzos requested a review from Copilot July 4, 2025 05:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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

@luoyuxia luoyuxia left a comment

Copy link
Copy Markdown
Contributor

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 HZY-Wade/fluss that referenced this pull request Aug 31, 2025
Ugbot pushed a commit to Ugbot/fluss that referenced this pull request Apr 26, 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.

3 participants