findingrish commented on code in PR #15817:
URL: https://github.com/apache/druid/pull/15817#discussion_r1562049787
##########
server/src/main/java/org/apache/druid/segment/metadata/CentralizedDatasourceSchemaConfig.java:
##########
@@ -29,36 +29,66 @@ public class CentralizedDatasourceSchemaConfig
{
@JsonProperty
private boolean enabled = false;
+ @JsonProperty
+ private boolean backFillEnabled = true;
+ @JsonProperty
+ private long backFillPeriod = 60000;
- // If realtime segment schema should be published in segment announcement
flow
- // This config is temporary and will be removed.
+ // internal config meant for testing
@JsonProperty
- private boolean announceRealtimeSegmentSchema = false;
+ private boolean taskSchemaPublishDisabled = false;
+ @JsonProperty
public boolean isEnabled()
{
return enabled;
}
- public boolean announceRealtimeSegmentSchema()
+ @JsonProperty
+ public boolean isBackFillEnabled()
+ {
+ return backFillEnabled;
+ }
+
+ @JsonProperty
+ public long getBackFillPeriod()
{
- return announceRealtimeSegmentSchema;
+ return backFillPeriod;
+ }
+
+ @JsonProperty
+ public boolean isTaskSchemaPublishDisabled()
+ {
+ return taskSchemaPublishDisabled;
}
public static CentralizedDatasourceSchemaConfig create()
{
return new CentralizedDatasourceSchemaConfig();
}
+ public static CentralizedDatasourceSchemaConfig create(boolean enabled)
Review Comment:
This is maintained per cluster. It name after the feature
`CentralizedDatasourceSchema`.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
