Details
-
Improvement
-
Status: Open
-
Major
-
Resolution: Unresolved
-
None
-
None
-
None
Description
RexLiteral may be sub-classed and some methods may be overridden. e.g. printAsJava. But currently, RexLiteral fields are private, sub-classes can't access these fields, which makes methods like printAsJava meaningless to be overridden. We recommend make RexLiteral member fields protected, so that we can sub-class it, and customize the behavior of methods such as printAsJava. Does it make sense?