yifan-c commented on code in PR #198: URL: https://github.com/apache/cassandra-sidecar/pull/198#discussion_r1970471506
########## CHANGES.txt: ########## @@ -8,7 +9,7 @@ * Sidecar schema initialization can be executed on multiple thread (CASSSIDECAR-200) * Make sidecar operations resilient to down Cassandra nodes (CASSSIDECAR-201) * Fix Cassandra instance not found error (CASSSIDECAR-192) - * Implemented Schema Reporter for Integration with DataHub (CASSSIDECAR-191) + * Implement Schema Reporter for Integration with DataHub (CASSSIDECAR-191) Review Comment: It is mainly avoid amending the changes log. It is not really a typo. "Implemented something" described the change as clear. If you look at the log, there are places where "Adds" and "Add" are used. There is no strict rule on the verb's form. -- 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: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org
