smiklosovic commented on code in PR #2458:
URL: https://github.com/apache/cassandra/pull/2458#discussion_r1257927646
##########
src/java/org/apache/cassandra/locator/AbstractCloudMetadataServiceConnector.java:
##########
@@ -61,6 +80,7 @@ public String apiCall(String url,
conn = (HttpURLConnection) new URL(url + query).openConnection();
extraHeaders.forEach(conn::setRequestProperty);
conn.setRequestMethod(method);
+ conn.setConnectTimeout(30000);
Review Comment:
well it is tricky to pass anything to these connectors from the properties
as currently it is accepting metadata service url only. We might either
refactor bunch of stuff for such a triviality or we can just have a system
property in `CassandraRelevantProperties`? It is hard to believe this would be
something which would be tweaked by users regularly, maybe for a power users as
the ultimate way to weak it but otherwise ....
--
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]
