Code taste / maintainability suggestion (non-blocking):\n\n logs full restored checkpoint records at INFO. These records may contain many pending file paths / partition values, so this can be noisy and expose unnecessary path details. Also is always true because the field is initialized to an empty array.\n\nSuggestion: log only restored flag + record count at INFO, and either remove contents or move them to DEBUG.\n\nExample:\n\n\n\nI also ran under ; it passed.

Correction: the previous review body was mangled by shell Markdown expansion. Please use this version instead.

Code taste / maintainability suggestion (non-blocking):

GlutenStreamingFileWriterOperator.initializeNativeState() logs full restored checkpoint records at INFO. These records may contain many pending file paths / partition values, so this can be noisy and expose unnecessary path details. Also restoredCheckpointRecords != null is always true because the field is initialized to an empty array.

Suggestion: log only restored flag + record count at INFO, and either remove contents or move them to DEBUG.

Example:

LOG.info(
    "Restore native file writer state for operator {}, restored {}, records {}",
    getDescription(),
    context.isRestored(),
    restoredCheckpointRecords.length);

I also ran ../build/mvn -pl runtime,ut -DskipTests spotless:check under gluten-flink/; it passed.