You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@gluten.apache.org by ph...@apache.org on 2024/04/09 00:55:59 UTC

(incubator-gluten) branch main updated: [GLUTEN-5316][CORE] Add @Override annotation for some methods (#5317)

This is an automated email from the ASF dual-hosted git repository.

philo pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-gluten.git


The following commit(s) were added to refs/heads/main by this push:
     new 0560ddd7b [GLUTEN-5316][CORE] Add @Override annotation for some methods (#5317)
0560ddd7b is described below

commit 0560ddd7b2b04a6f435de4b8bdd802396b984e4f
Author: Qian Sun <qi...@gmail.com>
AuthorDate: Tue Apr 9 08:55:55 2024 +0800

    [GLUTEN-5316][CORE] Add @Override annotation for some methods (#5317)
---
 .../main/java/org/apache/gluten/memory/memtarget/TreeMemoryTargets.java  | 1 +
 .../main/java/org/apache/gluten/substrait/rel/ExtensionTableNode.java    | 1 +
 .../src/main/java/org/apache/gluten/substrait/rel/LocalFilesNode.java    | 1 +
 .../java/org/apache/gluten/vectorized/ArrowWritableColumnVector.java     | 1 +
 4 files changed, 4 insertions(+)

diff --git a/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/TreeMemoryTargets.java b/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/TreeMemoryTargets.java
index 9566e11a5..a8dc39ce3 100644
--- a/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/TreeMemoryTargets.java
+++ b/gluten-core/src/main/java/org/apache/gluten/memory/memtarget/TreeMemoryTargets.java
@@ -139,6 +139,7 @@ public class TreeMemoryTargets {
       return granted;
     }
 
+    @Override
     public List<Spiller> getNodeSpillers() {
       return spillers;
     }
diff --git a/gluten-core/src/main/java/org/apache/gluten/substrait/rel/ExtensionTableNode.java b/gluten-core/src/main/java/org/apache/gluten/substrait/rel/ExtensionTableNode.java
index 8da97fa12..f5cfd1da5 100644
--- a/gluten-core/src/main/java/org/apache/gluten/substrait/rel/ExtensionTableNode.java
+++ b/gluten-core/src/main/java/org/apache/gluten/substrait/rel/ExtensionTableNode.java
@@ -177,6 +177,7 @@ public class ExtensionTableNode implements SplitInfo {
     return this.preferredLocations;
   }
 
+  @Override
   public ReadRel.ExtensionTable toProtobuf() {
     ReadRel.ExtensionTable.Builder extensionTableBuilder = ReadRel.ExtensionTable.newBuilder();
     StringValue extensionTable =
diff --git a/gluten-core/src/main/java/org/apache/gluten/substrait/rel/LocalFilesNode.java b/gluten-core/src/main/java/org/apache/gluten/substrait/rel/LocalFilesNode.java
index 428266b73..cbcda72dd 100644
--- a/gluten-core/src/main/java/org/apache/gluten/substrait/rel/LocalFilesNode.java
+++ b/gluten-core/src/main/java/org/apache/gluten/substrait/rel/LocalFilesNode.java
@@ -120,6 +120,7 @@ public class LocalFilesNode implements SplitInfo {
   protected void processFileBuilder(
       ReadRel.LocalFiles.FileOrFiles.Builder fileBuilder, int index) {}
 
+  @Override
   public ReadRel.LocalFiles toProtobuf() {
     ReadRel.LocalFiles.Builder localFilesBuilder = ReadRel.LocalFiles.newBuilder();
     // The input is iterator, and the path is in the format of: Iterator:index.
diff --git a/gluten-data/src/main/java/org/apache/gluten/vectorized/ArrowWritableColumnVector.java b/gluten-data/src/main/java/org/apache/gluten/vectorized/ArrowWritableColumnVector.java
index 3b1049fca..4603cbc41 100644
--- a/gluten-data/src/main/java/org/apache/gluten/vectorized/ArrowWritableColumnVector.java
+++ b/gluten-data/src/main/java/org/apache/gluten/vectorized/ArrowWritableColumnVector.java
@@ -558,6 +558,7 @@ public final class ArrowWritableColumnVector extends WritableColumnVectorShim {
    * Returns the dictionary Id for rowId. This should only be called when the ColumnVector is
    * dictionaryIds. We have this separate method for dictionaryIds as per SPARK-16928.
    */
+  @Override
   public int getDictId(int rowId) {
     assert (dictionary == null);
     return accessor.getInt(rowId);


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@gluten.apache.org
For additional commands, e-mail: commits-help@gluten.apache.org