Skip to content

Commit 833f87c

Browse files
authored
Merge pull request #5524 from gchq/5522-Aggregator-Not-Allowed-When-Specified
Issue 5522: DefaultCompactionRunnerFactory doesn't allow java compactions with aggregators specified
2 parents d2867ec + d94b252 commit 833f87c

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

java/compaction/compaction-job-execution/src/main/java/sleeper/compaction/job/execution/DefaultCompactionRunnerFactory.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -51,14 +51,6 @@ public DefaultCompactionRunnerFactory(ObjectFactory objectFactory, Configuration
5151
public CompactionRunner createCompactor(CompactionJob job, TableProperties tableProperties) {
5252
DataEngine engine = tableProperties.getEnumValue(DATA_ENGINE, DataEngine.class);
5353
CompactionRunner runner = createRunnerForEngine(engine);
54-
55-
// Has an experimental DataFusion only iterator been specified? If so, make sure
56-
// we are using the DataFusion compactor
57-
if (DataEngine.AGGREGATION_ITERATOR_NAME.equals(job.getIteratorClassName()) && !(runner instanceof DataFusionCompactionRunner)) {
58-
throw new IllegalStateException("DataFusion-only iterator specified, but DataFusion compactor not selected for job ID "
59-
+ job.getId() + " table ID " + job.getTableId());
60-
}
61-
6254
LOGGER.info("Selecting {} compactor (language {}) for job ID {} table ID {}", runner.getClass().getSimpleName(), runner.implementationLanguage(), job.getId(), job.getTableId());
6355
return runner;
6456
}

0 commit comments

Comments
 (0)