ZihanLi58 commented on a change in pull request #3172:
URL: https://github.com/apache/incubator-gobblin/pull/3172#discussion_r548134662
##########
File path:
gobblin-api/src/main/java/org/apache/gobblin/configuration/ConfigurationKeys.java
##########
@@ -1038,6 +1038,12 @@
public static final String AVRO_SCHEMA_CHECK_STRATEGY_DEFAULT =
"org.apache.gobblin.util.schema_check.AvroSchemaCheckDefaultStrategy";
+ /**
+ * Configuration and constant vale for GobblinMetadataChangeEvent
+ */
+ public static final String LIST_DELIMITER_KEY = ",";
Review comment:
This value will also be used by extractor, which will set the offset
range of the data topic. So to decouple the extractor and gmce publisher, I put
it in the common place
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]