Skip to content

Commit

Permalink
Add logging of which FS is used to read manifest files
Browse files Browse the repository at this point in the history
  • Loading branch information
phet committed Aug 15, 2023
1 parent 8bcc765 commit 17ac752
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@
import java.util.Properties;
import java.util.stream.Collectors;

import lombok.extern.slf4j.Slf4j;
import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.FileSystem;
import org.apache.hadoop.fs.Path;
Expand All @@ -36,6 +37,7 @@
import org.apache.gobblin.dataset.IterableDatasetFinder;


@Slf4j
public class ManifestBasedDatasetFinder implements IterableDatasetFinder<ManifestBasedDataset> {

public static final String CONFIG_PREFIX = CopyConfiguration.COPY_PREFIX + ".manifestBased";
Expand All @@ -54,6 +56,7 @@ public ManifestBasedDatasetFinder(final FileSystem srcFs, Properties properties)
this.manifestReadFs = optManifestReadFsUriStr.isPresent()
? FileSystem.get(URI.create(optManifestReadFsUriStr.get()), new Configuration())
: srcFs;
log.info("using file system to read manifest files: '{}'", this.manifestReadFs.getUri());
} catch (final IOException | IllegalArgumentException e) {
throw new RuntimeException("unable to create manifest-loading FS at URI '" + optManifestReadFsUriStr + "'", e);
}
Expand Down

0 comments on commit 17ac752

Please sign in to comment.