Skip to content

Commit

Permalink
rethrow rle
Browse files Browse the repository at this point in the history
  • Loading branch information
LakshSingla committed Sep 12, 2023
1 parent 478a984 commit 05ece4c
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -769,8 +769,7 @@ private static <T, QueryType extends Query<T>> Optional<DataSource> materializeR
);

if (!framesOptional.isPresent()) {
log.debug("Unable to materialize the results as frames. Defaulting to materializing the results as rows");
return Optional.empty();
throw DruidException.defensive("Unable to materialize the results as frames. Defaulting to materializing the results as rows");
}

Sequence<FrameSignaturePair> frames = framesOptional.get();
Expand All @@ -792,6 +791,9 @@ private static <T, QueryType extends Query<T>> Optional<DataSource> materializeR
return Optional.of(new FrameBasedInlineDataSource(frameSignaturePairs, toolChest.resultArraySignature(query)));

}
catch (ResourceLimitExceededException e) {
throw e;
}
catch (UnsupportedColumnTypeException e) {
subqueryStatsProvider.incrementSubqueriesFallingBackDueToUnsufficientTypeInfo();
log.debug(e, "Type info in signature insufficient to materialize rows as frames.");
Expand Down

0 comments on commit 05ece4c

Please sign in to comment.