Skip to content

Commit

Permalink
Merge pull request #3431 from pnorbert/bpls-aws
Browse files Browse the repository at this point in the history
When we use AWSSDK, bpls should not check if the file  it exists loca…
  • Loading branch information
pnorbert authored Jan 11, 2023
2 parents 3dd524f + bacbf96 commit e64c720
Showing 1 changed file with 21 additions and 3 deletions.
24 changes: 21 additions & 3 deletions source/utils/bpls/bpls.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1533,10 +1533,28 @@ int doList(const char *path)
if (verbose > 1)
printf("\nADIOS Open: read header info from %s\n", path);

if (!adios2sys::SystemTools::FileExists(path))
std::string tpl = helper::LowerCase(transport_params);
bool remoteFile = (tpl.find("awssdk") != std::string::npos);
if (remoteFile)
{
fprintf(stderr, "\nError: input path %s does not exist\n", path);
return 4;
if (engine_name.empty())
{
fprintf(
stderr,
"\nError: For remote file access you must specify the engine "
"explicitly with -E parameter, e.g. -E bp5 or -E bp4 or -E "
"bp3.\nVirtual engines like BPFile or FileStream do not know "
"which engine to use.\n");
return 8;
}
}
else
{
if (!adios2sys::SystemTools::FileExists(path))
{
fprintf(stderr, "\nError: input path %s does not exist\n", path);
return 4;
}
}

// initialize BP reader
Expand Down

0 comments on commit e64c720

Please sign in to comment.