rgw/rest: add TODO for concurrent endpoint DNS resolution

Signed-off-by: Oguzhan Ozmen <oozmen@bloomberg.net>

diff --git a/src/rgw/rgw_rest_conn.cc b/src/rgw/rgw_rest_conn.cc
index ce3425f6e8c66180ca26cc8640392892611f6514..6beb262e84aa27edf615239c4d17b4ab248992f9 100644 (file)
--- a/src/rgw/rgw_rest_conn.cc
+++ b/src/rgw/rgw_rest_conn.cc
@@ -13,6 +13,11 @@
 using namespace std;
 
 void RGWRESTConn::resolve_endpoints() {
+  // TODO: resolve endpoints concurrently using async_resolve on a shared io_context.
+  //       In practice zones have a single endpoint or at most a handful of individual
+  //       RGW instances, and resolution runs once at process startup, so sequential
+  //       cost is negligible. A natural time to address this would be when adding
+  //       periodic DNS re-resolution.
   for (auto& res_ep : resolved_endpoints) {
     const std::string& ep_url = res_ep.url;
     // parse URL