[2146] Code review changes

s3/s3_operations.cpp

@@ -425,7 +425,7 @@ namespace irods_s3 {
                 const auto s3_key_name = fmt::format("/{}/{}/{}", bucket_name, obj_id, object_name);
 
                 // Update physical path but only on first creation otherwise the policy that was in effect
-                // at the time the object was first created rules. 
+                // at the time the object was first created wins.
                 if (L1desc[index].openType == CREATE_TYPE) {
                     logger::debug("{}:{} ({}) [[{}]] updating physical_path to {}",
                             __FILE__, __LINE__, __FUNCTION__, thread_id, s3_key_name.c_str());
@@ -2271,8 +2271,9 @@ namespace irods_s3 {
 
             // Must update the physical_path in the L1desc[] table for decoupled naming.
             // In the case of a redirect, this runs on the original connected server and this
-            // is the server that updates the database. This will open update for creates.
-            // Anything else uses whatever was previously in the database.
+            // is the server that updates the database. In update_physical_path_for_decoupled_naming,
+            // the update will only happen if it is a create. Anything else uses whatever was
+            // previously in the database.
             update_physical_path_for_decoupled_naming(_ctx);
         }
         return SUCCESS();