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) {