HBASE-25627: HBase replication should have a metric to represent if the source is stuck getting initialized#3018
Conversation
…he source is stuck getting initialized
c8a6577 to
062cf2f
Compare
|
@apurtell @bharathv @wchevreuil, this is a PR following your review comments here. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
bharathv
left a comment
There was a problem hiding this comment.
just a few nits, lgtm otherwise.
...in/java/org/apache/hadoop/hbase/replication/regionserver/MetricsReplicationSourceSource.java
Show resolved
Hide resolved
...in/java/org/apache/hadoop/hbase/replication/regionserver/MetricsReplicationSourceSource.java
Outdated
Show resolved
Hide resolved
...server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSource.java
Show resolved
Hide resolved
...er/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSource.java
Outdated
Show resolved
Hide resolved
|
💔 -1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
💔 -1 overall
This message was automatically generated. |
bharathv
left a comment
There was a problem hiding this comment.
Couple of nits, but lgtm.
hbase-hadoop-compat/src/main/java/org/apache/hadoop/metrics2/lib/DynamicMetricsRegistry.java
Outdated
Show resolved
Hide resolved
...ava/org/apache/hadoop/hbase/replication/regionserver/MetricsReplicationSourceSourceImpl.java
Outdated
Show resolved
Hide resolved
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
@wchevreuil do you mind reviewing this one? This is after your review here |
xcangCRM
left a comment
There was a problem hiding this comment.
2 nit comments, Ship it! 👍
...g/apache/hadoop/hbase/replication/regionserver/MetricsReplicationGlobalSourceSourceImpl.java
Show resolved
Hide resolved
hbase-hadoop-compat/src/main/java/org/apache/hadoop/metrics2/lib/DynamicMetricsRegistry.java
Outdated
Show resolved
Hide resolved
...er/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSource.java
Outdated
Show resolved
Hide resolved
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
These metrics keep track of sources getting initialized, and if for any reason source is stuck during initialization, monitoring and alerting can be built on top of that.