[CALCITE-2462] RexProgramTest: move "rex building" methods to base class

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.17.0
• Fix Version/s: 1.18.0
• Component/s: core
• Assignee: Julian Hyde
• Reporter: Vladimir Sitnikov
• Created: 10/Aug/18 18:29
• Updated: 23/Dec/18 00:12
• Resolved: 20/Aug/18 08:06

Description

RexProgramTest is quite big (2000 lines now), and "easy to use" rex building is useful for many tests.

So I suggest to move methods like gt, lt, etc to the base class, so multiple other tests can reuse it.

Alternative option would be to use Kotlin for builders, yet I think Java tests will be there for quite a while, so it would be nice to simplify them as well.

Issue Links

• links to: PR 778