Fix errorprone and Javadoc warnings

data-loader/core/src/main/java/com/scalar/db/dataloader/core/ColumnInfo.java

@@ -9,6 +9,7 @@
  * <p>This class holds the metadata for a column, including the namespace (schema), table name, and
  * the column name within the table.
  */
+@SuppressWarnings("SameNameButDifferent")
 @Value
 @Builder
 public class ColumnInfo {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/ScanRange.java

@@ -4,6 +4,7 @@
 import lombok.Value;
 
 /** * The scan range which is used in data export scan filtering */
+@SuppressWarnings("SameNameButDifferent")
 @Value
 public class ScanRange {
   /** The key for scan start filter */

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataexport/ExportManager.java

@@ -28,6 +28,7 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+@SuppressWarnings({"SameNameButDifferent", "FutureReturnValueIgnored"})
 @RequiredArgsConstructor
 public abstract class ExportManager {
   private static final Logger logger = LoggerFactory.getLogger(ExportManager.class);

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataexport/ExportOptions.java

@@ -9,8 +9,8 @@
 import lombok.Builder;
 import lombok.Data;
 
-/** Options for a ScalarDB export data operation */
-@SuppressWarnings("SameNameButDifferent")
+/** Options for a ScalarDB export data operation. */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Builder(builderMethodName = "hiddenBuilder")
 @Data
 public class ExportOptions {
@@ -31,6 +31,15 @@ public class ExportOptions {
   @Builder.Default private List<String> projectionColumns = Collections.emptyList();
   private List<Scan.Ordering> sortOrders;
 
+  /**
+   * Generates and returns an export options builder.
+   *
+   * @param namespace namespaces for export
+   * @param tableName tableName for export
+   * @param scanPartitionKey scan partition key for export
+   * @param outputFileFormat output file format for export
+   * @return a configured export options builder
+   */
   public static ExportOptionsBuilder builder(
       String namespace, String tableName, Key scanPartitionKey, FileFormat outputFileFormat) {
     return hiddenBuilder()

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataexport/producer/JsonLineProducerTask.java

@@ -16,13 +16,10 @@
 import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class JsonLineProducerTask extends ProducerTask {
 
   private final DataLoaderObjectMapper objectMapper = new DataLoaderObjectMapper();
-  private static final Logger logger = LoggerFactory.getLogger(JsonLineProducerTask.class);
 
   /**
    * Class constructor

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataexport/producer/JsonProducerTask.java

@@ -17,14 +17,11 @@
 import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class JsonProducerTask extends ProducerTask {
 
   private final DataLoaderObjectMapper objectMapper = new DataLoaderObjectMapper();
   private final boolean prettyPrintJson;
-  private static final Logger logger = LoggerFactory.getLogger(JsonProducerTask.class);
 
   /**
    * Class constructor

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataexport/producer/ProducerTaskFactory.java

@@ -8,6 +8,7 @@
 import java.util.Map;
 import lombok.RequiredArgsConstructor;
 
+@SuppressWarnings("SameNameButDifferent")
 @RequiredArgsConstructor
 public class ProducerTaskFactory {
 

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataexport/validation/ExportOptionsValidator.java

@@ -17,6 +17,7 @@
  * A validator for ensuring that export options are consistent with the ScalarDB table metadata and
  * follow the defined constraints.
  */
+@SuppressWarnings("SameNameButDifferent")
 @NoArgsConstructor(access = AccessLevel.PRIVATE)
 public class ExportOptionsValidator {
 

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/ImportManager.java

@@ -34,6 +34,7 @@
  *   <li>Notifying listeners of various import events
  * </ul>
  */
+@SuppressWarnings("SameNameButDifferent")
 @AllArgsConstructor
 public class ImportManager implements ImportEventListener {
 

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/ImportOptions.java

@@ -7,7 +7,8 @@
 import lombok.Builder;
 import lombok.Data;
 
-/** Import options to import data into one or more ScalarDB tables */
+/** Import options to import data into one or more ScalarDB tables. */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Builder
 @Data
 public class ImportOptions {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/controlfile/ControlFile.java

@@ -11,6 +11,7 @@
  * Represents a control file that holds control file tables which contains the column mappings that
  * maps a source file column to the actual database table column.
  */
+@SuppressWarnings("SameNameButDifferent")
 @Getter
 @Setter
 public class ControlFile {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/controlfile/ControlFileTable.java

@@ -12,6 +12,7 @@
  * table name, and field mappings. This class is used to define how data from a control file maps to
  * a specific table in ScalarDB.
  */
+@SuppressWarnings("SameNameButDifferent")
 @Getter
 @Setter
 public class ControlFileTable {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/controlfile/ControlFileTableFieldMapping.java

@@ -10,6 +10,7 @@
  * This class defines how data from a specific field in the input source should be mapped to the
  * corresponding column in the database.
  */
+@SuppressWarnings("SameNameButDifferent")
 @Getter
 @Setter
 public class ControlFileTableFieldMapping {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/datachunk/ImportDataChunk.java

@@ -4,7 +4,8 @@
 import lombok.Builder;
 import lombok.Data;
 
-/** * Import data chunk data */
+/** * Import data chunk data. */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Data
 @Builder
 public class ImportDataChunk {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/datachunk/ImportDataChunkStatus.java

@@ -6,7 +6,8 @@
 import lombok.Builder;
 import lombok.Data;
 
-/** * A DTO to store import data chunk details */
+/** * A DTO to store import data chunk details. */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Data
 @Builder
 @JsonDeserialize(builder = ImportDataChunkStatus.ImportDataChunkStatusBuilder.class)

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/datachunk/ImportRow.java

@@ -3,7 +3,8 @@
 import com.fasterxml.jackson.databind.JsonNode;
 import lombok.Value;
 
-/** Stores data related to a single row on import file */
+/** Stores data related to a single row on import file. */
+@SuppressWarnings("SameNameButDifferent")
 @Value
 public class ImportRow {
   int rowNumber;

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/log/AbstractImportLogger.java

@@ -24,6 +24,7 @@
  * logging transaction batch results and managing event listeners. Concrete implementations should
  * define how to log transaction batches and handle errors.
  */
+@SuppressWarnings("SameNameButDifferent")
 @RequiredArgsConstructor
 public abstract class AbstractImportLogger implements ImportEventListener {
 

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/log/ImportLoggerConfig.java

@@ -10,6 +10,7 @@
  */
 @Value
 @Builder
+@SuppressWarnings("SameNameButDifferent")
 public class ImportLoggerConfig {
   /**
    * The directory path where log files will be stored. This path should end with a directory

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/log/writer/DefaultLogWriterFactory.java

@@ -10,6 +10,7 @@
  * it creates. It's annotated with Lombok's {@code @AllArgsConstructor} to automatically generate a
  * constructor that initializes the configuration field.
  */
+@SuppressWarnings("SameNameButDifferent")
 @AllArgsConstructor
 public class DefaultLogWriterFactory implements LogWriterFactory {
 

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/processor/ImportProcessor.java

@@ -41,11 +41,12 @@
  * supports both transactional and non-transactional (storage) modes and provides event notification
  * capabilities for monitoring the import process.
  */
+@SuppressWarnings("SameNameButDifferent")
 @RequiredArgsConstructor
 public abstract class ImportProcessor {
 
   final ImportProcessorParams params;
-  private static final Logger LOGGER = LoggerFactory.getLogger(ImportProcessor.class);
+  private static final Logger logger = LoggerFactory.getLogger(ImportProcessor.class);
   private final List<ImportEventListener> listeners = new ArrayList<>();
 
   /**
@@ -316,13 +317,13 @@ private ImportTransactionBatchResult processTransactionBatch(
 
     } catch (TransactionException e) {
       isSuccess = false;
-      LOGGER.error(e.getMessage());
+      logger.error(e.getMessage());
       try {
         if (transaction != null) {
           transaction.abort(); // Ensure transaction is aborted
         }
       } catch (TransactionException abortException) {
-        LOGGER.error(
+        logger.error(
             "Failed to abort transaction: {}", abortException.getMessage(), abortException);
       }
       error = e.getMessage();

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/processor/ImportProcessorParams.java

@@ -17,6 +17,7 @@
  * <p>This class is immutable and uses the Builder pattern for construction. It encapsulates all
  * required parameters and dependencies for processing data imports in ScalarDB.
  */
+@SuppressWarnings("SameNameButDifferent")
 @Builder
 @Value
 public class ImportProcessorParams {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/task/ImportTask.java

@@ -38,6 +38,7 @@
  * functionality to import data into single or multiple tables based on the provided import options
  * and control file configurations.
  */
+@SuppressWarnings({"SameNameButDifferent"})
 @RequiredArgsConstructor
 public abstract class ImportTask {
 
@@ -148,7 +149,8 @@ private List<ImportTargetResult> startMultiTableImportProcess(
               copyNode);
       targetResults.add(result);
     }
-    return targetResults;
+    // Wrapped in unmodifiable list to fix MixedMutabilityReturnType error-prone warning
+    return Collections.unmodifiableList(targetResults);
   }
 
   /**

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/task/ImportTaskParams.java

@@ -14,6 +14,7 @@
  * Parameters required for executing an import task in the data loader. This class encapsulates all
  * necessary information needed to process and import a single record into ScalarDB.
  */
+@SuppressWarnings("SameNameButDifferent")
 @Builder
 @Value
 public class ImportTaskParams {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/task/ImportTransactionalTask.java

@@ -3,8 +3,6 @@
 import com.scalar.db.api.DistributedTransaction;
 import com.scalar.db.api.Result;
 import com.scalar.db.dataloader.core.dataimport.dao.ScalarDbDaoException;
-import com.scalar.db.exception.transaction.AbortException;
-import com.scalar.db.exception.transaction.TransactionException;
 import com.scalar.db.io.Column;
 import com.scalar.db.io.Key;
 import java.util.List;
@@ -83,24 +81,4 @@ protected void saveRecord(
       throws ScalarDbDaoException {
     params.getDao().put(namespace, tableName, partitionKey, clusteringKey, columns, transaction);
   }
-
-  /**
-   * Aborts the active ScalarDB transaction if it has not been committed.
-   *
-   * <p>This method provides a safe way to abort an active transaction, handling any abort-related
-   * exceptions by wrapping them in a {@link TransactionException}.
-   *
-   * @param tx the transaction to be aborted. If null, this method does nothing
-   * @throws TransactionException if an error occurs during the abort operation or if the underlying
-   *     abort operation fails
-   */
-  private void abortActiveTransaction(DistributedTransaction tx) throws TransactionException {
-    if (tx != null) {
-      try {
-        tx.abort();
-      } catch (AbortException e) {
-        throw new TransactionException(e.getMessage(), tx.getId());
-      }
-    }
-  }
 }

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/task/result/ImportTargetResult.java

@@ -6,6 +6,8 @@
 import lombok.Builder;
 import lombok.Value;
 
+/** To store import target result. */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Builder
 @Value
 public class ImportTargetResult {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/task/result/ImportTaskResult.java

@@ -7,6 +7,7 @@
 import lombok.Builder;
 import lombok.Value;
 
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Builder
 @Value
 @JsonDeserialize(builder = ImportTaskResult.ImportTaskResultBuilder.class)

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/task/validation/ImportSourceRecordValidationResult.java

@@ -31,17 +31,29 @@ public void addErrorMessage(String columnName, String errorMessage) {
     this.errorMessages.add(errorMessage);
   }
 
-  /** @return Immutable list of validation error messages */
+  /**
+   * Return error messages list.
+   *
+   * @return Immutable list of validation error messages.
+   */
   public List<String> getErrorMessages() {
     return Collections.unmodifiableList(this.errorMessages);
   }
 
-  /** @return Immutable set of columns that had errors */
+  /**
+   * A set of columns with errors is stored and returned.
+   *
+   * @return Immutable set of columns that had errors.
+   */
   public Set<String> getColumnsWithErrors() {
     return Collections.unmodifiableSet(this.columnsWithErrors);
   }
 
-  /** @return Validation is valid or not */
+  /**
+   * Stores validation result.
+   *
+   * @return Validation is valid or not.
+   */
   public boolean isValid() {
     return this.errorMessages.isEmpty();
   }

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/task/validation/ImportSourceRecordValidator.java

@@ -9,6 +9,7 @@
 import lombok.AccessLevel;
 import lombok.NoArgsConstructor;
 
+@SuppressWarnings("SameNameButDifferent")
 @NoArgsConstructor(access = AccessLevel.PRIVATE)
 public class ImportSourceRecordValidator {
 

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/transactionbatch/ImportTransactionBatch.java

@@ -6,6 +6,7 @@
 import lombok.Value;
 
 /** Transaction batch details */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Builder
 @Value
 public class ImportTransactionBatch {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/transactionbatch/ImportTransactionBatchResult.java

@@ -8,6 +8,7 @@
 import lombok.Value;
 
 /** Transaction batch result */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Builder
 @Value
 @JsonDeserialize(builder = ImportTransactionBatchResult.ImportTransactionBatchResultBuilder.class)

data-loader/core/src/main/java/com/scalar/db/dataloader/core/dataimport/transactionbatch/ImportTransactionBatchStatus.java

@@ -5,7 +5,8 @@
 import lombok.Builder;
 import lombok.Value;
 
-/** Batch status details */
+/** Batch status details. */
+@SuppressWarnings({"SameNameButDifferent", "MissingSummary"})
 @Builder
 @Value
 public class ImportTransactionBatchStatus {

data-loader/core/src/main/java/com/scalar/db/dataloader/core/tablemetadata/TableMetadataRequest.java

@@ -3,6 +3,7 @@
 import lombok.Getter;
 
 /** Represents the request for metadata for a single ScalarDB table */
+@SuppressWarnings("SameNameButDifferent")
 @Getter
 public class TableMetadataRequest {