Skip to content
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

Postgres support #100

Merged
merged 5 commits into from
Aug 14, 2024
Merged

Postgres support #100

merged 5 commits into from
Aug 14, 2024

Conversation

dmgaldi
Copy link
Contributor

@dmgaldi dmgaldi commented Aug 9, 2024

No description provided.

@dmgaldi dmgaldi requested a review from ryanrdoherty August 9, 2024 13:47
@dmgaldi dmgaldi self-assigned this Aug 9, 2024
@@ -166,6 +166,11 @@
<scope>test</scope>
</dependency>

<dependency>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since we're manually adding the postgres driver into tomcat, can you remove this?

@@ -162,7 +162,7 @@ private void insertInstanceRow(InstanceInfo instanceInfo) throws WdkModelExcepti
Types.VARCHAR, // table name
Types.VARCHAR, // query name
Types.VARCHAR, // checksum
Types.CLOB // result message
Types.OTHER // result message
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Curious about what led to this choice. Can we use platform functions to do this? Looks like we just have schema gen and writing methods, but I think that's because writing the value can be the same on both DBs? Maybe with a different column type in postgres?

}
catch (SQLRunnerException e) {
throw WdkModelException.translateFrom(e);
}
}

private String getInsertUserRefSql() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will have to take your word for it on this one. :)

@dmgaldi dmgaldi merged commit 0f0d09e into j21tc9 Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants