Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 commented on PR #8162: URL: https://github.com/apache/hbase/pull/8162#issuecomment-4342648114 Failed UT are not related, let me merge -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 merged PR #8162: URL: https://github.com/apache/hbase/pull/8162 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 merged PR #8161: URL: https://github.com/apache/hbase/pull/8161 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 merged PR #8160: URL: https://github.com/apache/hbase/pull/8160 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 commented on PR #8160: URL: https://github.com/apache/hbase/pull/8160#issuecomment-4342632876 Falied UT are not related, Let me merge -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 closed pull request #8160: HBASE-30126 Fix mismatched LoggerFactory class references URL: https://github.com/apache/hbase/pull/8160 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
Copilot commented on code in PR #8162:
URL: https://github.com/apache/hbase/pull/8162#discussion_r3158528272
##
hbase-server/src/main/java/org/apache/hadoop/hbase/wal/AbstractRecoveredEditsOutputSink.java:
##
@@ -47,7 +47,7 @@
@InterfaceAudience.Private
abstract class AbstractRecoveredEditsOutputSink extends OutputSink {
- private static final Logger LOG =
LoggerFactory.getLogger(RecoveredEditsOutputSink.class);
+ private static final Logger LOG =
LoggerFactory.getLogger(AbstractRecoveredEditsOutputSink.class);
Review Comment:
This logger declaration exceeds the 100-character Checkstyle LineLength
limit. Please wrap the assignment across multiple lines to avoid Checkstyle
failures (max=100).
```suggestion
private static final Logger LOG =
LoggerFactory.getLogger(AbstractRecoveredEditsOutputSink.class);
```
##
hbase-server/src/test/java/org/apache/hadoop/hbase/tool/BulkLoadHFilesSplitRecoveryTestBase.java:
##
@@ -86,7 +86,7 @@
*/
public class BulkLoadHFilesSplitRecoveryTestBase {
- private static final Logger LOG =
LoggerFactory.getLogger(TestHRegionServerBulkLoad.class);
+ private static final Logger LOG =
LoggerFactory.getLogger(BulkLoadHFilesSplitRecoveryTestBase.class);
Review Comment:
This logger declaration exceeds the 100-character Checkstyle LineLength
limit. Please wrap the assignment across multiple lines (as done in other
updated logger declarations) to keep the line length <= 100.
```suggestion
private static final Logger LOG =
LoggerFactory.getLogger(BulkLoadHFilesSplitRecoveryTestBase.class);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
[PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 opened a new pull request, #8162: URL: https://github.com/apache/hbase/pull/8162 - for branch-2.5 - see: HBASE-30126 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
[PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 opened a new pull request, #8161: URL: https://github.com/apache/hbase/pull/8161 - for branch-2.6 - see: HBASE-30126 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
[PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 opened a new pull request, #8160: URL: https://github.com/apache/hbase/pull/8160 - for branch-2 - see: HBASE-30126 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 merged PR #8152: URL: https://github.com/apache/hbase/pull/8152 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
Copilot commented on code in PR #8152:
URL: https://github.com/apache/hbase/pull/8152#discussion_r3148821198
##
hbase-server/src/main/java/org/apache/hadoop/hbase/tool/CanaryTool.java:
##
@@ -798,7 +798,7 @@ public Void call() {
private static final long DEFAULT_TIMEOUT = 60; // 10 mins
private static final int MAX_THREADS_NUM = 16; // #threads to contact regions
- private static final Logger LOG = LoggerFactory.getLogger(Canary.class);
+ private static final Logger LOG = LoggerFactory.getLogger(CanaryTool.class);
Review Comment:
This changes the SLF4J logger category from `Canary` to `CanaryTool`, which
can break existing operator log configuration (e.g., log4j properties targeting
the old category) and result in missing expected logs. If logger category
stability is important for this tool, consider preserving the previous category
(e.g., via a string logger name) or documenting the category change in release
notes/migration guidance.
```suggestion
private static final Logger LOG = LoggerFactory.getLogger("Canary");
```
##
hbase-client/src/main/java/org/apache/hadoop/hbase/client/RawAsyncHBaseAdmin.java:
##
@@ -379,7 +379,7 @@ class RawAsyncHBaseAdmin implements AsyncAdmin {
public static final String FLUSH_TABLE_PROCEDURE_SIGNATURE =
"flush-table-proc";
- private static final Logger LOG =
LoggerFactory.getLogger(AsyncHBaseAdmin.class);
+ private static final Logger LOG =
LoggerFactory.getLogger(RawAsyncHBaseAdmin.class);
Review Comment:
This updates the logger category from `AsyncHBaseAdmin` to
`RawAsyncHBaseAdmin`. If downstream deployments filter/route client logs by the
previous category, this can change logging behavior unexpectedly. If the intent
is purely to fix mismatched references while preserving loggers consumed
externally, consider keeping the previous category (e.g., using the old class
or a constant string name) or calling out this compatibility impact explicitly.
```suggestion
private static final Logger LOG =
LoggerFactory.getLogger(AsyncHBaseAdmin.class);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
Copilot commented on code in PR #8152:
URL: https://github.com/apache/hbase/pull/8152#discussion_r3148425811
##
hbase-server/src/main/java/org/apache/hadoop/hbase/wal/AbstractRecoveredEditsOutputSink.java:
##
@@ -46,7 +46,7 @@
@InterfaceAudience.Private
abstract class AbstractRecoveredEditsOutputSink extends OutputSink {
- private static final Logger LOG =
LoggerFactory.getLogger(RecoveredEditsOutputSink.class);
+ private static final Logger LOG =
LoggerFactory.getLogger(AbstractRecoveredEditsOutputSink.class);
Review Comment:
This logger declaration is likely to exceed typical checkstyle max line
length used in HBase modules. Please wrap to match the prevalent style in this
PR (e.g., split assignment and `LoggerFactory.getLogger(...)` across lines) to
avoid checkstyle failures.
```suggestion
private static final Logger LOG =
LoggerFactory.getLogger(AbstractRecoveredEditsOutputSink.class);
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
[PR] HBASE-30126 Fix mismatched LoggerFactory class references [hbase]
liuxiaocs7 opened a new pull request, #8152: URL: https://github.com/apache/hbase/pull/8152 - see: HBASE-30126 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
