prateekm commented on a change in pull request #987: SAMZA-2158: Remove the
redunant coordinator stream reads in the ApplicationMaster startup sequence.
URL: https://github.com/apache/samza/pull/987#discussion_r273150796
##########
File path:
samza-core/src/main/java/org/apache/samza/container/LocalityManager.java
##########
@@ -45,32 +39,26 @@
private final MetadataStore metadataStore;
/**
- * Builds the LocalityManager based upon {@link Config} and {@link
MetricsRegistry}.
- * Uses the {@link CoordinatorStreamValueSerde} to serialize messages before
- * reading/writing into metadata store.
+ * <ul>
Review comment:
Please format as a paragraph instead of a list everywhere. It's easier to
read when looking at the code.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
