You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by hy...@apache.org on 2020/07/17 17:29:38 UTC

[calcite] branch master updated: [CALCITE-4127] Remove final from AbstractRelNode#getRelTypeName

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

hyuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/master by this push:
     new cdf8678  [CALCITE-4127] Remove final from AbstractRelNode#getRelTypeName
cdf8678 is described below

commit cdf8678f33f8ea49e5291fce4a4651a24edaf19c
Author: rubenada <ru...@gmail.com>
AuthorDate: Fri Jul 17 10:00:52 2020 +0200

    [CALCITE-4127] Remove final from AbstractRelNode#getRelTypeName
---
 core/src/main/java/org/apache/calcite/rel/AbstractRelNode.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/org/apache/calcite/rel/AbstractRelNode.java b/core/src/main/java/org/apache/calcite/rel/AbstractRelNode.java
index b55c90d..7a528d5 100644
--- a/core/src/main/java/org/apache/calcite/rel/AbstractRelNode.java
+++ b/core/src/main/java/org/apache/calcite/rel/AbstractRelNode.java
@@ -179,7 +179,8 @@ public abstract class AbstractRelNode implements RelNode {
     Util.discard(planner);
   }
 
-  public final String getRelTypeName() {
+  // It is not recommended to override this method, but sub-classes can do it at their own risk.
+  public String getRelTypeName() {
     String cn = getClass().getName();
     int i = cn.length();
     while (--i >= 0) {