Skip to content

Commit

Permalink
addressed PR + fix toc async read
Browse files Browse the repository at this point in the history
  • Loading branch information
danovaro committed Nov 29, 2024
1 parent 99d130a commit e5f2ace
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/fdb5/io/LustreSettings.cc
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ bool fdb5LustreapiSupported() {
#endif
}

int fdb5LustreapiFileCreate(eckit::PathName path, LustreStripe stripe) {
int fdb5LustreapiFileCreate(const eckit::PathName& path, LustreStripe stripe) {

#if defined(fdb5_HAVE_LUSTRE)

Expand Down
2 changes: 1 addition & 1 deletion src/fdb5/io/LustreSettings.h
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ LustreStripe stripeDataLustreSettings();

//----------------------------------------------------------------------------------------------------------------------

int fdb5LustreapiFileCreate(eckit::PathName path, LustreStripe stripe);
int fdb5LustreapiFileCreate(const eckit::PathName& path, LustreStripe stripe);

bool fdb5LustreapiSupported();

Expand Down
5 changes: 2 additions & 3 deletions src/fdb5/toc/TocHandler.cc
Original file line number Diff line number Diff line change
Expand Up @@ -725,7 +725,7 @@ class SubtocPreloader {
int fd;
SYSCALL2((fd = ::open(path.localPath(), iomode)), path);
closers.emplace_back(AutoFDCloser{fd});
eckit::Length tocSize = 2*1024*1024;
eckit::Length tocSize = path.size();

aiocb& aio(aiocbs[i]);
zero(aio);
Expand Down Expand Up @@ -802,9 +802,8 @@ class SubtocPreloader {
}
}
#else
NOTIMP
NOTIMP;
#endif // eckit_HAVE_AIO

return std::move(subTocReadCache_);
}

Expand Down

0 comments on commit e5f2ace

Please sign in to comment.