Removed the use of H5VLpeek_connector_id_by_name

.github/workflows/ubuntu_mpich.yml

@@ -33,7 +33,7 @@ jobs:
         - name: Set up dependencies
           run: |
             sudo apt-get update
-            sudo apt-get install automake autoconf libtool libtool-bin m4
+            sudo apt-get install automake autoconf libtool libtool-bin m4 uuid-dev
             # The MPICH installed on github actions is too slow
             # sudo apt-get install mpich
             # mpicc -v
@@ -103,7 +103,7 @@ jobs:
             autoreconf -i
             ./configure --with-hdf5=${GITHUB_WORKSPACE}/HDF5 \
                         --enable-test-netcdf4=${GITHUB_WORKSPACE}/NetCDF \
-                        --enable-test-hdf5-iotest \
+                        --disable-test-hdf5-iotest \
                         --with-mpi=${GITHUB_WORKSPACE}/MPICH
             # Do NOT test QMCPACK, as it requires FFTW which fails to build
         - name: Print config.log if error

.github/workflows/ubuntu_openmpi.yml

@@ -32,7 +32,7 @@ jobs:
         - name: Set up dependencies
           run: |
             sudo apt-get update
-            sudo apt-get install automake autoconf libtool libtool-bin m4
+            sudo apt-get install automake autoconf libtool libtool-bin m4 uuid-dev
             # zlib
             sudo apt-get install zlib1g-dev
             # mpi
@@ -98,7 +98,7 @@ jobs:
             autoreconf -i
             ./configure --with-hdf5=${GITHUB_WORKSPACE}/HDF5 \
                         --enable-test-netcdf4=${GITHUB_WORKSPACE}/NetCDF \
-                        --enable-test-hdf5-iotest \
+                        --disable-test-hdf5-iotest \
                         TESTMPIRUN="mpiexec --oversubscribe -n NP"
             # Do NOT test QMCPACK, as it requires FFTW which fails to build
         - name: Print config.log if error

src/H5VL_log_file.cpp

@@ -88,7 +88,7 @@ void *H5VL_log_file_create (
             htri_t ret;
             ret = H5VLis_connector_registered_by_name ("native");
             if (ret != 1) { ERR_OUT ("Native VOL not found") }
-            uvlid = H5VLpeek_connector_id_by_name ("native");
+            uvlid = H5VLget_connector_id_by_name("native");
             CHECK_ID (uvlid)
             under_vol_info = NULL;
         }
@@ -173,6 +173,10 @@ void *H5VL_log_file_create (
         H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VLFILE_CREATE);
         H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VL_LOG_FILE_CREATE_FILE);
         H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VL_LOG_FILE_CREATE);
+        if (!info) {
+            err = H5VLclose(uvlid);
+            CHECK_ERR
+        }
     }
     H5VL_LOGI_EXP_CATCH
 
@@ -236,7 +240,7 @@ void *H5VL_log_file_open (
             htri_t ret;
             ret = H5VLis_connector_registered_by_name ("native");
             if (ret != 1) { ERR_OUT ("Native VOL not found") }
-            uvlid = H5VLpeek_connector_id_by_name ("native");
+            uvlid = H5VLget_connector_id_by_name("native");
             CHECK_ID (uvlid)
             under_vol_info = NULL;
             // return NULL;
@@ -304,6 +308,12 @@ void *H5VL_log_file_open (
         H5VL_log_filei_register (fp);
 
         H5VL_LOGI_PROFILING_TIMER_STOP (fp, TIMER_H5VL_LOG_FILE_OPEN);
+
+        if (!info) {
+            err = H5VLclose(uvlid);
+            CHECK_ERR
+        }
+
     }
     H5VL_LOGI_EXP_CATCH
 
@@ -430,7 +440,7 @@ herr_t H5VL_log_file_specific (void *file,
                     htri_t ret;
                     ret = H5VLis_connector_registered_by_name ("native");
                     if (ret != 1) { ERR_OUT ("Native VOL not found") }
-                    uvlid = H5VLpeek_connector_id_by_name ("native");
+                    uvlid = H5VLget_connector_id_by_name("native");
                     CHECK_ID (uvlid)
                     under_vol_info = NULL;
                 }
@@ -440,6 +450,10 @@ herr_t H5VL_log_file_specific (void *file,
                 err = H5VLfile_specific (NULL, uvlid, args, dxpl_id, req);
                 CHECK_ERR
                 H5Pclose (under_fapl_id);
+                if (!info) {
+                   err = H5VLclose(uvlid);
+                   CHECK_ERR
+                }
             } break;
             case H5VL_FILE_FLUSH: {
                 H5VL_LOGI_PROFILING_TIMER_START;

utils/h5ldump/h5ldump.cpp

@@ -32,6 +32,7 @@ const char *hdf5sig = "\211HDF\r\n\032\n";
 const char ncsig[] = {'C', 'D', 'F'};
 
 inline std::string get_file_signature (std::string &path) {
+    herr_t err = 0;
     hid_t fid        = -1;  // File ID
     hid_t faplid     = -1;  // File access property ID
     hid_t nativevlid = -1;  // Native VOL ID
@@ -50,9 +51,14 @@ inline std::string get_file_signature (std::string &path) {
         fin.close ();
         if (!memcmp (hdf5sig, sig, 8)) {
             // Always use native VOL
-            nativevlid = H5VLpeek_connector_id_by_name ("native");
+            nativevlid = H5VLget_connector_id_by_name("native");
+            CHECK_ID (nativevlid)
             faplid     = H5Pcreate (H5P_FILE_ACCESS);
-            H5Pset_vol (faplid, nativevlid, NULL);
+            CHECK_ID (faplid)
+            err = H5Pset_vol (faplid, nativevlid, NULL);
+            CHECK_ERR
+            err = H5VLclose(nativevlid);
+            CHECK_ERR
 
             // Open the input file
             fid = H5Fopen (path.c_str (), H5F_ACC_RDONLY, faplid);
@@ -215,9 +221,15 @@ void h5ldump_file (std::string path,
     });
 
     // Always use native VOL
-    nativevlid = H5VLpeek_connector_id_by_name ("native");
+    nativevlid = H5VLget_connector_id_by_name("native");
+    CHECK_ID (nativevlid)
     faplid     = H5Pcreate (H5P_FILE_ACCESS);
-    H5Pset_vol (faplid, nativevlid, NULL);
+    CHECK_ID (faplid)
+    err = H5Pset_vol (faplid, nativevlid, NULL);
+    CHECK_ERR
+    err = H5VLclose(nativevlid);
+    CHECK_ERR
+
 
     // Open the input file
     fid = H5Fopen (path.c_str (), H5F_ACC_RDONLY, faplid);

utils/h5ldump/h5ldump_visit.cpp

@@ -38,9 +38,14 @@ void h5ldump_visit (std::string path, std::vector<H5VL_log_dset_info_t> &dsets)
     });
 
     // Always use native VOL
-    nativevlid = H5VLpeek_connector_id_by_name ("native");
+    nativevlid = H5VLget_connector_id_by_name("native");
+    CHECK_ID (nativevlid)
     faplid     = H5Pcreate (H5P_FILE_ACCESS);
-    H5Pset_vol (faplid, nativevlid, NULL);
+    CHECK_ID (faplid)
+    err = H5Pset_vol (faplid, nativevlid, NULL);
+    CHECK_ERR
+    err = H5VLclose(nativevlid);
+    CHECK_ERR
 
     // Open the input file
     fid = H5Fopen (path.c_str (), H5F_ACC_RDONLY, faplid);

utils/h5lreplay/h5lreplay.cpp

@@ -176,16 +176,19 @@ void h5lreplay_core (std::string &inpath, std::string &outpath, int rank, int np
     });
 
     // Open the input and output file
-    nativevlid = H5VLpeek_connector_id_by_name ("native");
+    nativevlid = H5VLget_connector_id_by_name("native");
+    CHECK_ID (nativevlid)
     faplid     = H5Pcreate (H5P_FILE_ACCESS);
     CHECK_ID (faplid)
     err = H5Pset_fapl_mpio (faplid, MPI_COMM_WORLD, MPI_INFO_NULL);
     CHECK_ERR
     err = H5Pset_vol (faplid, nativevlid, NULL);
     CHECK_ERR
+    err = H5VLclose(nativevlid);
+    CHECK_ERR
 
     finid = H5Fopen (inpath.c_str (), H5F_ACC_RDONLY, faplid);
-    CHECK_ID (faplid)
+    CHECK_ID (finid)
     foutid = H5Fcreate (outpath.c_str (), H5F_ACC_TRUNC, H5P_DEFAULT, faplid);
     CHECK_ID (foutid)
     mpierr = MPI_File_open (MPI_COMM_WORLD, inpath.c_str (), MPI_MODE_RDONLY, MPI_INFO_NULL, &fin);