dcapwell commented on code in PR #3865:
URL: https://github.com/apache/cassandra/pull/3865#discussion_r1940272420
##########
src/java/org/apache/cassandra/service/accord/fastpath/ParameterizedFastPathStrategy.java:
##########
@@ -146,7 +146,7 @@ static WeightedDc fromString(String s, int idx)
else if (parts.length == 2)
return new WeightedDc(validateDC(parts[0]),
validateWeight(parts[1]), false);
else
- throw cfe("Invalid dc weighting syntax %s, use <dc>:<weight>");
+ throw cfe("Invalid dc weighting syntax %s, use <dc>:<weight>",
s);
Review Comment:
this was the error we saw in UI, and its because `foo:bar:42`... the `dc`
had `:` in the name, so hit this logic, but since `, s` was missing it threw an
issue with string formatting rather than the CFE we expected
--
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]
