Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ local-dev/
!gradle/wrapper/gradle-wrapper.jar
!**/src/main/**
!**/src/test/**
*.csv

### STS ###
.apt_generated
Expand Down
2 changes: 2 additions & 0 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,8 @@ dependencies {
implementation group: 'org.webjars', name: 'swagger-ui', version: '3.24.0'
implementation group: 'org.aspectj', name: 'aspectjweaver', version: '1.8.8'
implementation group: 'com.google.cloud', name: 'google-cloud-bigquery', version: '1.124.7'
implementation group: 'com.google.code.gson', name: 'gson', version: '2.8.5'
implementation group: 'com.jayway.jsonpath', name: 'json-path', version: '2.4.0'

// These dependencies are required to load the logback config file.
implementation group: 'org.codehaus.groovy', name: 'groovy', version: '3.0.7'
Expand Down
29 changes: 29 additions & 0 deletions jsonOutput-8.csv

Large diffs are not rendered by default.

3 changes: 3 additions & 0 deletions src/main/java/bio/terra/cda/app/aop/TimerAspect.java
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
package bio.terra.cda.app.aop;

import org.aspectj.lang.ProceedingJoinPoint;

import org.aspectj.lang.Signature;
import org.aspectj.lang.annotation.Around;
import org.aspectj.lang.annotation.Aspect;
Expand All @@ -19,8 +20,10 @@ public Object executionTimer(ProceedingJoinPoint joinPoint) throws Throwable {
long start = System.currentTimeMillis();
Object proceed = joinPoint.proceed();
long executionTime = System.currentTimeMillis() - start;

Signature signature = joinPoint.getSignature();
logger.info("--Execution Timer: {} executed in {}ms", signature, executionTime);

return proceed;
}
}
17 changes: 14 additions & 3 deletions src/main/java/bio/terra/cda/app/controller/QueryApiController.java
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,15 @@
import bio.terra.cda.generated.model.QueryCreatedData;
import bio.terra.cda.generated.model.QueryResponseData;
import com.google.cloud.bigquery.BigQueryException;

import java.io.IOException;
import java.net.MalformedURLException;
import java.net.URL;
import java.util.Collections;
import java.util.Set;
import javax.servlet.http.HttpServletRequest;
import javax.validation.Valid;

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.annotation.Autowired;
Expand Down Expand Up @@ -63,9 +72,11 @@ private String createNextUrl(String jobId, int offset, int limit) {

@TrackExecutionTime
@Override
public ResponseEntity<QueryResponseData> query(String id, Integer offset, Integer limit) {
var result = queryService.getQueryResults(id, offset, limit);
var response =
public ResponseEntity<QueryResponseData> query(
String id, Integer offset, Integer limit, String format, String includeHeaders) {
QueryService.QueryResult result =
queryService.getQueryResults(id, offset, limit, format, includeHeaders);
QueryResponseData response =
new QueryResponseData()
.result(Collections.unmodifiableList(result.items))
.totalRowCount(result.totalRowCount)
Expand Down
Loading