Skip to content

Commit

Permalink
update test case for JoinDataSource since it has a new JoinAlgorithm …
Browse files Browse the repository at this point in the history
…field now
  • Loading branch information
cecemei committed Dec 18, 2024
1 parent c7dc930 commit af51747
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ public DataSource withChildren(List<DataSource> children)
throw new IAE("Expected [1] child, got [%d]", children.size());
}

return RestrictedDataSource.create(children.get(0), rowFilter);
return create(children.get(0), rowFilter);
}

@Override
Expand Down Expand Up @@ -151,7 +151,7 @@ public Function<SegmentReference, SegmentReference> createSegmentMapFunction(
@Override
public DataSource withUpdatedDataSource(DataSource newSource)
{
return RestrictedDataSource.create(newSource, rowFilter);
return create(newSource, rowFilter);
}

@Override
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -516,7 +516,8 @@ public void testGetAnalysisWithRestrictedDS()
JoinType.LEFT,
null,
ExprMacroTable.nil(),
null
null,
JoinAlgorithm.BROADCAST
);
DataSourceAnalysis analysis = dataSource.getAnalysis();
Assert.assertEquals("table1", analysis.getBaseDataSource().getTableNames().iterator().next());
Expand Down

0 comments on commit af51747

Please sign in to comment.