Hi community. Recently, when I review CALCITE-5769(https://github.com/apache/calcite/pull/3296), I found something a little confusing.
First, the java doc in RexProgramBuilderBase.vParamNotNull(), which meant to create non-nullable variable, but it’s returning description is “nullable varchar variable”. Second, we use vDecimal(int arg) to create nullable decimal variable, but the RelDataType we pass in is “nonNullableDecimal”, which I think should be “nullableDecimal”. So does the other vXxx() methods. I’m not sure if I understand right here. If it’s something we can improve, I’ll file a JIRA case to record and fix it. Best regards, Zhe Hu
