Skip to content

Commit

Permalink
Merge pull request #3344 from ingef/feature/stats-for-internal-form
Browse files Browse the repository at this point in the history
allow Statistics for all SingleTableResults
  • Loading branch information
awildturtok authored Mar 18, 2024
2 parents 6dc0964 + 2f9d5a6 commit 19e17ed
Showing 1 changed file with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@
import com.bakdata.conquery.models.execution.ExecutionState;
import com.bakdata.conquery.models.execution.ManagedExecution;
import com.bakdata.conquery.models.query.ManagedQuery;
import com.bakdata.conquery.models.query.SingleTableResult;
import io.dropwizard.auth.Auth;
import io.dropwizard.jersey.PATCH;
import lombok.RequiredArgsConstructor;
Expand Down Expand Up @@ -62,8 +63,8 @@ public FullExecutionStatus getStatus(@Auth Subject subject, @PathParam(QUERY) Ma
@Path("{" + QUERY + "}/statistics")
public Response getDescription(@Auth Subject subject, @PathParam(QUERY) ManagedExecution query) {

if (!(query instanceof ManagedQuery)) {
throw new BadRequestException("Statistics is only available for %s".formatted(ManagedQuery.class.getSimpleName()));
if (!(query instanceof SingleTableResult)) {
throw new BadRequestException("Statistics is only available for %s".formatted(SingleTableResult.class.getSimpleName()));
}

subject.authorize(query.getDataset(), Ability.READ);
Expand Down

0 comments on commit 19e17ed

Please sign in to comment.