mynameborat commented on code in PR #1640:
URL: https://github.com/apache/samza/pull/1640#discussion_r1017221725
##########
samza-core/src/main/java/org/apache/samza/config/ApplicationConfig.java:
##########
@@ -46,6 +46,11 @@ public enum ApplicationMode {
BATCH
}
+ public enum ApplicationApiType {
+ LEGACY,
+ LOW_LEVEL,
+ HIGH_LEVEL
+ }
Review Comment:
The reason to do so even if its internally used is to make sure any changes
to this enum should be treated as API changes and would need to be treated as
backward compatible as this will be used in the coordinator stream
configuration which are persisted and read across different versions of samza.
--
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]
