yifan-c commented on code in PR #171:
URL: https://github.com/apache/cassandra-sidecar/pull/171#discussion_r1928035653
##########
server-common/src/main/java/org/apache/cassandra/sidecar/common/server/StorageOperations.java:
##########
@@ -62,6 +61,8 @@ void takeSnapshot(@NotNull String tag, @NotNull String
keyspace, @NotNull String
* @param keyspace keyspace to check the data ownership; Cassandra selects
a keyspace if null value is passed.
* @return ring view
* @throws UnknownHostException when hostname of peer Cassandra nodes
cannot be resolved
+ *
+ * TODO: refactor. Do not return http response payload object from this
layer.
Review Comment:
true. I will file a jira
--
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]
