[CALCITE-3322] There are two same testcase in RelMetadataTest

• Type: Test
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.20.0
• Fix Version/s: 1.21.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: shenh062326
• Created: 04/Sep/19 02:54
• Updated: 12/Sep/19 16:27
• Resolved: 06/Sep/19 12:50

Description

When I read the code, I found two identical testcases in RelMetadataTest, except for the different names, which might need to be changed to limit 0, or just need to keep one.

  @Test public void testRowCountSortLimit() {
    final String sql = "select * from emp order by ename limit 10";
    checkRowCount(sql, 10d, 0D, 10d);
  }

  @Test public void testRowCountSortLimit0() {
    final String sql = "select * from emp order by ename limit 10";
    checkRowCount(sql, 10d, 0D, 10d);
  }

Issue Links

• links to: GitHub Pull Request #1439