Fix issue with S3 FS not working with segments with spaces in the name

pinot-core/src/main/java/org/apache/pinot/core/data/manager/BaseTableDataManager.java

@@ -378,6 +378,13 @@ public void reloadSegment(String segmentName, IndexLoadingConfig indexLoadingCon
         // doesn't load anything from an empty indexDir, but gets the info to complete the copyTo.
         try (SegmentDirectory segmentDirectory = initSegmentDirectory(segmentName, String.valueOf(zkMetadata.getCrc()),
             indexLoadingConfig)) {
+          if (segmentDirectory.isDirectoryMatchingWithLatestConfigs()) {
+            // The segment directory is already in sync with latest configs
+            ImmutableSegment segment = ImmutableSegmentLoader.load(indexDir, indexLoadingConfig, schema);
+            addSegment(segment);
+            removeBackup(indexDir);
+            return;
+          }
           segmentDirectory.copyTo(indexDir);
         }
       }

pinot-plugins/pinot-file-system/pinot-s3/src/main/java/org/apache/pinot/plugin/filesystem/S3PinotFS.java

@@ -204,9 +204,9 @@ private String normalizeToDirectoryPrefix(URI uri)
     Preconditions.checkNotNull(uri, "uri is null");
     URI strippedUri = getBase(uri).relativize(uri);
     if (isPathTerminatedByDelimiter(strippedUri)) {
-      return sanitizePath(strippedUri.getPath());
+      return sanitizePath(strippedUri.toString());
     }
-    return sanitizePath(strippedUri.getPath() + DELIMITER);
+    return sanitizePath(strippedUri.toString() + DELIMITER);
   }
 
   private URI normalizeToDirectoryUri(URI uri)

pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/store/SegmentLocalFSDirectory.java

@@ -101,6 +101,11 @@ public SegmentLocalFSDirectory(File directoryFile, SegmentMetadataImpl metadata,
     }
   }
 
+  @Override
+  public boolean isDirectoryMatchingWithLatestConfigs() {
+    return false;
+  }
+
   @Override
   public URI getIndexDir() {
     return _indexDir.toURI();

pinot-segment-spi/src/main/java/org/apache/pinot/segment/spi/store/SegmentDirectory.java

@@ -98,6 +98,8 @@
  */
 public abstract class SegmentDirectory implements Closeable {
 
+  public abstract boolean isDirectoryMatchingWithLatestConfigs();
+
   public abstract URI getIndexDir();
 
   public abstract SegmentMetadataImpl getSegmentMetadata();