Jie Cheng created CALCITE-6543:
----------------------------------

             Summary: Change the RelOptCostImpl toString method to be 
consistent with VolcanoCost
                 Key: CALCITE-6543
                 URL: https://issues.apache.org/jira/browse/CALCITE-6543
             Project: Calcite
          Issue Type: Improvement
          Components: core
    Affects Versions: 1.37.0
            Reporter: Jie Cheng
             Fix For: 1.37.0


Hello everyone, I am a calcite novice, recently learning calcite source code, 
in the process of learning, found that RelOptCostImpl toString method is 
inconsistent with the VolcanoCost method, I think we should revise them to be 
similar

before modification:
{quote}@Override public String toString() {
if (value == Double.MAX_VALUE) {
return "huge";
} else {
return Double.toString(value);
}
}
{quote}
after modification:
{quote}@Override public String toString() {
if (value == Double.MAX_VALUE) {
return "huge";
} else if (value == Double.POSITIVE_INFINITY) {
return "inf";
} else if (value == 1.0) {
return "tiny";
} else if (value == 0.0) {
return "0";
} else {
return Double.toString(value);
}
}
{quote}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to