virajjasani commented on code in PR #2191:
URL: https://github.com/apache/phoenix/pull/2191#discussion_r2145819854
##########
phoenix-core-server/src/main/java/org/apache/phoenix/coprocessor/PhoenixMasterObserver.java:
##########
@@ -82,7 +82,8 @@ public class PhoenixMasterObserver implements MasterObserver,
MasterCoprocessor
= "UPSERT INTO " + SYSTEM_CDC_STREAM_NAME + " (TABLE_NAME,
STREAM_NAME, PARTITION_ID, "
+ "PARENT_PARTITION_ID, PARTITION_END_TIME) VALUES (?,?,?,?,?)";
- private static final int MAX_RETRY_COUNT = 20;
+ public static final String PHOENIX_MASTER_MAX_RETRY_COUNT =
"phoenix.master.max.retry.count";
Review Comment:
I doubt if we use `observer` in any configs, Observer is internal coproc
concept. Let me check if we use it in config keys already.
--
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: issues-unsubscr...@phoenix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
