-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor dao classes to extract param build, row insert #1001
Conversation
dexamundsen
commented
Sep 12, 2024
•
edited
Loading
edited
- Will help reuse them in clone cohort
- also contains changes from PR comments : Util to generateId, remove extra userEmail args #999
.addValue("enum", enumValue); | ||
} | ||
|
||
MapSqlParameterSource buildValueParam( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
made them package private to be callable from other dao classes
.addValue("data_type", annotationKey.getDataType().name()); | ||
int rowsAffected = jdbcTemplate.update(sql, params); | ||
LOGGER.debug("CREATE annotation key rowsAffected = {}", rowsAffected); | ||
MapSqlParameterSource keyParamSets = |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
general pattern: build param, then insert rows
@@ -64,7 +64,7 @@ jobs: | |||
env: | |||
TEST_PROJECT_SA_KEY: ${{ secrets.TEST_PROJECT_SA_KEY }} | |||
- name: Gradle Run Regression Tests Only | |||
run: ./gradlew service:regressionTests -PregressionTestUnderlays=cmssynpuf,aouSR2019q4r4 --scan | |||
run: ./gradlew service:regressionTests -PregressionTestUnderlays=cmssynpuf,aouSR2019q4r4 --info --scan |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A comparison of without - https://github.com/DataBiosphere/tanagra/actions/runs/10859538266/job/30139008330#step:8:1
and with -
https://github.com/DataBiosphere/tanagra/actions/runs/10857889854/job/30135081427#step:8:1
For now keeping it out since all of the extra logs are less about the application tests etc and more about the gradle build process, which is not needed regularly.
keeping logs concide really helps with debugging
# Conflicts: # service/src/main/java/bio/terra/tanagra/db/CohortDao.java