Commit 86ad40c
fix(db) loadSubset when orderby has multiple columns (#926)
* fix(db) loadSubset when orderby has multiple columns
failing test for multiple orderby and loadsubset
push down multiple orderby predicates to load subset
split order by cursor predicate build into two, inprecise wider band for local lading, precise for the sync loadSubset
new e2e tests for composite orderby and pagination
changeset
when doing gt/lt comparisons to a bool cast to string
fix: use non-boolean columns in multi-column orderBy e2e tests
Electric/PostgreSQL doesn't support comparison operators (<, >, <=, >=)
on boolean types. Changed tests to use age (number) and name (string)
columns instead of isActive (boolean) to avoid this limitation.
The core multi-column orderBy functionality still works correctly -
this is just a test adjustment to work within Electric's SQL parser
constraints.
* ci: apply automated fixes
---------
Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>1 parent 09da081 commit 86ad40c
File tree
9 files changed
+1366
-86
lines changed- .changeset
- packages
- db-collection-e2e/src/suites
- db
- src
- collection
- query
- compiler
- live
- utils
- tests
- query
- electric-db-collection/src
9 files changed
+1366
-86
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
0 commit comments