Renaming

Author: ldematte

java/cuvs-java/src/main/java22/com/nvidia/cuvs/internal/CagraIndexImpl.java

@@ -414,8 +414,8 @@ public void serializeToHNSW(OutputStream outputStream, Path tempFile, int buffer
     checkNotDestroyed();
     tempFile = tempFile.toAbsolutePath();
 
-    try (var arena = Arena.ofConfined()) {
-      MemorySegment pathSeg = buildMemorySegment(arena, tempFile.toString());
+    try (var localArena = Arena.ofConfined()) {
+      MemorySegment pathSeg = buildMemorySegment(localArena, tempFile.toString());
 
       long cuvsRes = resources.getHandle();
       int returnValue =
@@ -611,18 +611,19 @@ public static CagraIndex merge(CagraIndex[] indexes, CagraMergeParams mergeParam
     var mergedIndex = createCagraIndex();
     long cuvsRes = resources.getHandle();
 
-    try (var arena = Arena.ofConfined()) {
+    try (var localArena = Arena.ofConfined()) {
       MemorySegment indexesSegment =
-          arena.allocate(indexes.length * ValueLayout.ADDRESS.byteSize());
+          localArena.allocate(indexes.length * ValueLayout.ADDRESS.byteSize());
 
       for (int i = 0; i < indexes.length; i++) {
         CagraIndexImpl indexImpl = (CagraIndexImpl) indexes[i];
         indexesSegment.setAtIndex(
             ValueLayout.ADDRESS, i, indexImpl.cagraIndexReference.getMemorySegment());
       }
 
-      // TODO: IMO we should call cuvsCreateMergeParams here, instead of allocating this ourselves
-      var mergeParamsSegment = createMergeParamsSegment(arena, mergeParams);
+      // TODO: we should call cuvsCreateMergeParams here, instead of allocating this ourselves
+      // See https://github.com/rapidsai/cuvs/pull/1109
+      var mergeParamsSegment = createMergeParamsSegment(localArena, mergeParams);
 
       var returnValue =
           cuvsCagraMerge(cuvsRes, mergeParamsSegment, indexesSegment, indexes.length, mergedIndex);