Skip to content

Commit

Permalink
Sync new Release/0008 with Master (#168)
Browse files Browse the repository at this point in the history
* [ALS-5422] Add persistence.xml to visualization resource (#161)

The visualization resource is failing to start due to an error injecting persistence unit into CDI managed bean. It is unable to find a persistence unit named ''.

* [ALS-0000] Fix aggregate resource

- Very confused as to how this was working before

* [ALS-5422] Add persistence.xml to visualization resource (#161) (#162) (#163)

The visualization resource is failing to start due to an error injecting persistence unit into CDI managed bean. It is unable to find a persistence unit named ''.

---------

Co-authored-by: Luke Sikina <lucas.skina@gmail.com>
  • Loading branch information
Gcolon021 and Luke Sikina authored Dec 8, 2023
1 parent 7f645ee commit 86275ce
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,8 @@ public Response querySync(QueryRequest queryRequest) {

Set<String> allowedResultTypes = Set.of(
"COUNT", "CROSS_COUNT", "INFO_COLUMN_LISTING", "OBSERVATION_COUNT",
"OBSERVATION_CROSS_COUNT", "CATEGORICAL_CROSS_COUNT", "CONTINUOUS_CROSS_COUNT"
"OBSERVATION_CROSS_COUNT", "CATEGORICAL_CROSS_COUNT", "CONTINUOUS_CROSS_COUNT",
"VARIANT_COUNT_FOR_QUERY", "AGGREGATE_VCF_EXCERPT", "VCF_EXCERPT"
);

if (!allowedResultTypes.contains(expectedResultType)) {
Expand Down

0 comments on commit 86275ce

Please sign in to comment.