-
Notifications
You must be signed in to change notification settings - Fork 6
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(deps): bump sqldelight from 2.0.1 to 2.0.2 (WPB-10501) #2714
base: develop
Are you sure you want to change the base?
Conversation
Bumps `sqldelight` from 2.0.1 to 2.0.2. Updates `app.cash.sqldelight:runtime` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:coroutines-extensions` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:android-driver` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:androidx-paging3-extensions` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:native-driver` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:sqlite-driver` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:web-worker-driver` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:primitive-adapters` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:sqlite-3-38-dialect` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:postgresql-dialect` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:r2dbc-driver` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight:async-extensions` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) Updates `app.cash.sqldelight` from 2.0.1 to 2.0.2 - [Release notes](https://github.com/cashapp/sqldelight/releases) - [Changelog](https://github.com/cashapp/sqldelight/blob/master/CHANGELOG.md) - [Commits](sqldelight/sqldelight@2.0.1...2.0.2) --- updated-dependencies: - dependency-name: app.cash.sqldelight:runtime dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:coroutines-extensions dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:android-driver dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:androidx-paging3-extensions dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:native-driver dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:sqlite-driver dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:web-worker-driver dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:primitive-adapters dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:sqlite-3-38-dialect dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:postgresql-dialect dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:r2dbc-driver dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight:async-extensions dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: app.cash.sqldelight dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
6d71a92
to
9c2eecd
Compare
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.
There are broken tests in MessageNotificationsTest
with this upgrade, related to notifications queries (row_num) So we cannot update until those are fixed.
Quality Gate passedIssues Measures |
Test Results740 tests - 2 410 718 ✔️ - 2 327 4m 20s ⏱️ +40s For more details on these failures, see this check. Results for commit 756d615. ± Comparison against base commit be8472c. This pull request removes 3150 and adds 740 tests. Note that renamed tests count towards both.
This pull request removes 105 skipped tests and adds 12 skipped tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Datadog ReportBranch report: ❌ 9 Failed (0 Known Flaky), 2913 Passed, 52 Skipped, 12.37s Total Time ❌ Failed Tests (9)
|
Seems to be a bug within SqlDelight. We will need a 2.0.3 build. So, I'll investigate further. This is the query that is facing issues: getNotificationsMessages:
WITH NumberedMessages AS (
SELECT
-- other fields
ROW_NUMBER() OVER (PARTITION BY m.conversation_id ORDER BY m.creation_date DESC) AS row_num
FROM
Message m
JOIN
User u ON m.sender_user_id = u.qualified_id
JOIN
Conversation c ON m.conversation_id = c.qualified_id
LEFT JOIN
MessageAssetContent ac ON m.id = ac.message_id AND m.conversation_id = ac.conversation_id
LEFT JOIN
MessageTextContent tc ON m.id = tc.message_id AND m.conversation_id = tc.conversation_id
WHERE
-- Criteria
)
SELECT
-- some fields
FROM
NumberedMessages
WHERE
row_num <= 10
ORDER BY
date DESC; However, take a look into the WITH NumberedMessages AS (
SELECT
-- more fields
row_num -- It completely removed the rest of the query here
FROM
Message m
JOIN
User u ON m.sender_user_id = u.qualified_id
JOIN
Conversation c ON m.conversation_id = c.qualified_id
LEFT JOIN |
Datadog ReportBranch report: ✅ 0 Failed, 1 Passed, 0 Skipped, 0s Total Time |
Quality Gate passedIssues Measures |
Bencher
Click to view all benchmark results
Bencher - Continuous Benchmarking View Public Perf Page Docs | Repo | Chat | Help |
This PR is stale because it has been open 30 days with no activity. Please update it or close it in case is not relevant anymore. |
Bumps
sqldelight
from 2.0.1 to 2.0.2.Updates
app.cash.sqldelight:runtime
from 2.0.1 to 2.0.2Release notes
Sourced from app.cash.sqldelight:runtime's releases.
... (truncated)
Changelog
Sourced from app.cash.sqldelight:runtime's changelog.
Commits
a82c184
Prepare for release 2.0.23a778a7
Inline the column names for SELECT * (#5054)ebabf09
Use the extensions array instead of stream (#5127)b372df0
Bump intellij version (#4983)ad70ba1
Dont bundle the kotlin lib into the intellij plugin as its already a plugin d...b5d9d3f
Fixes 5032 column adjacency for PostgreSql UPDATE FROM statement (#5035)aa41c0b
Specify the thread to use for the connection list clear action (#5125)2c54d9e
Update plugin ksp to v1.9.23-1.0.20 (#5121)918fc32
Update build.gradle (#5118)5628209
Fixes 5082 tsvector (#5104)Updates
app.cash.sqldelight:coroutines-extensions
from 2.0.1 to 2.0.2Release notes
Sourced from app.cash.sqldelight:coroutines-extensions's releases.
... (truncated)
Changelog
Sourced from app.cash.sqldelight:coroutines-extensions's changelog.
Commits
a82c184
Prepare for release 2.0.23a778a7
Inline the column names for SELECT * (#5054)ebabf09
Use the extensions array instead of stream (#5127)b372df0
Bump intellij version (#4983)ad70ba1
Dont bundle the kotlin lib into the intellij plugin as its already a plugin d...b5d9d3f
Fixes 5032 column adjacency for PostgreSql UPDATE FROM statement (#5035)aa41c0b
Specify the thread to use for the connection list clear action (#5125)2c54d9e
Update plugin ksp to v1.9.23-1.0.20 (#5121)918fc32
Update build.gradle (#5118)5628209
Fixes 5082 tsvector (#5104)Updates
app.cash.sqldelight:android-driver
from 2.0.1 to 2.0.2Release notes
Sourced from app.cash.sqldelight:android-driver's releases.
... (truncated)
Changelog
Sourced from app.cash.sqldelight:android-driver's changelog.
Commits
a82c184
Prepare for release 2.0.23a778a7
Inline the column names for SELECT * (#5054)ebabf09
Use the extensions array instead of stream (#5127)b372df0
Bump intellij version (#4983)ad70ba1
Dont bundle the kotlin lib into the intellij plugin as its already a plugin d...b5d9d3f
Fixes 5032 column adjacency for PostgreSql UPDATE FROM statement (#5035)aa41c0b
Specify the thread to use for the connection list clear action (#5125)2c54d9e
Update plugin ksp to v1.9.23-1.0.20 (#5121)918fc32
Update build.gradle (#5118)5628209
Fixes 5082 tsvector (#5104)Updates
app.cash.sqldelight:androidx-paging3-extensions
from 2.0.1 to 2.0.2Release notes
Sourced from app.cash.sqldelight:androidx-paging3-extensions's releases.
... (truncated)
Changelog
Sourced from app.cash.sqldelight:androidx-paging3-extensions's changelog.
Commits
a82c184
Prepare for release 2.0.23a778a7
Inline the column names for SELECT * (#5054)ebabf09
Use the extensions array instead of stream (#5127)b372df0
Bump intellij version (#4983)ad70ba1
Dont bundle the kotlin lib into the intellij plugin as its already a plugin d...b5d9d3f
Fixes 5032 column adjacency for PostgreSql UPDATE FROM statement (#5035)aa41c0b
Specify the thread to use for the connection list clear action (#5125)2c54d9e
Update plugin ksp to v1.9.23-1.0.20 (#5121)918fc32
Update build.gradle (#5118)5628209
Fixes 5082 tsvector (#5104)Updates
app.cash.sqldelight:native-driver
from 2.0.1 to 2.0.2Release notes
Sourced from app.cash.sqldelight:native-driver's releases.
... (truncated)
Changelog
Sourced from app.cash.sqldelight:native-driver's changelog.