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

chore: postgres configuration in jvm for global db (do not merge) (WPB-9012) #2758

Closed
wants to merge 15 commits into from

Conversation

yamilmedina
Copy link
Contributor

@yamilmedina yamilmedina commented May 13, 2024

TaskWPB-9012 [Kalium] Support pg in Kalium JVM for globaldb - ANSI SQL

NOTES

DO NOT MERGE, just for validation purposes.

This uses ANSI SQL, and unit tests running testcontainers.

@yamilmedina yamilmedina added WIP work in progress DO NOT MERGE labels May 13, 2024
@yamilmedina yamilmedina changed the title chore: validate running postgres in jvm for global db (do not merge) (WPB-9012) chore: postgres configuration in jvm for global db (do not merge) (WPB-9012) May 13, 2024
Copy link
Contributor

github-actions bot commented May 13, 2024

Test Results

3 038 tests  ±0   2 934 ✔️ ±0   4m 27s ⏱️ +50s
   528 suites ±0      104 💤 ±0 
   528 files   ±0          0 ±0 

Results for commit 71f24db. ± Comparison against base commit f8c7a9f.

♻️ This comment has been updated with latest results.

@datadog-wireapp
Copy link

datadog-wireapp bot commented May 13, 2024

Datadog Report

Branch report: chore/validate-pg
Commit report: 73e74cf
Test service: kalium-jvm

✅ 0 Failed, 2934 Passed, 104 Skipped, 11s Total Time

Comment on lines +131 to +132
<ID>ArgumentListWrapping:GlobalDBBaseTest.kt$GlobalDBBaseTest$( PlatformDatabaseData(StorageData.FileBacked(storePath)), StandardTestDispatcher(), null, false )</ID>
<ID>ArgumentListWrapping:GlobalDatabase.kt$( identifier = Int.MIN_VALUE, sql = """SELECT table_name FROM information_schema.tables where table_name = 'accounts'""", mapper = { if (it.next().value) { val result = it.getString(0) println("THE RESULT: $result") app.cash.sqldelight.db.QueryResult.Value(result?.isNotEmpty()) } else { app.cash.sqldelight.db.QueryResult.Value(false) } }, parameters = 0 )</ID>
Copy link
Member

Choose a reason for hiding this comment

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

I know this is a draft, but are new lines to baseline.xml intended?

Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

Copy link
Contributor Author

@yamilmedina yamilmedina left a comment

Choose a reason for hiding this comment

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

.

Copy link
Contributor Author

@yamilmedina yamilmedina left a comment

Choose a reason for hiding this comment

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

.

@yamilmedina
Copy link
Contributor Author

Closing as we are not going to move forward with this approach

@vitorhugods vitorhugods deleted the chore/validate-pg branch January 29, 2025 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants