Jack, could you provide some reasoning why your new TimestampArithmeticFnNode node doesn't fall into the existing heirachy for arithmetic operators, namely BinaryArithmeticOperatorNode?
I would like to see Derby consolidate more compilation nodes, rather than create new ones. New nodes increase the static and runtime footprint, but don't usually add much value if the logic could be in an exisiting node. > +public class TimestampArithmeticFnNode extends ValueNode Dan.
