Outer `where` on an inner-joined subquery source is ignored (matches every row)
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 65/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- javascript, typescript
- Lĩnh vực
- backend-api-design, databases
Hướng nghiên cứu
The bug is in the query builder's join handling when the source is a subquery. Start by examining the join logic in the query builder files, likely in packages/db-core/src/query/query.ts or similar. The repro script shows the exact failing case: inner join with a subquery source. Look at how the where clause is applied to joined aliases and compare the handling for collection vs subquery sources. Run the repro script to confirm the fix works for all eight combinations.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
- I've validated the bug against the latest version of DB packages (
@tanstack/db@0.9.2, currentlylatest)
Describe the bug
When a live query inner-joins a subquery (a new Query().from(...).where(...) used as the join source), a where on the joined alias in the outer query is ignored. The outer query returns every joined row, as if the where weren't there.
- The subquery's own
whereis still applied. Only the outerwhereon the joined alias is lost. - Every neighbouring shape works:
- joining the bare collection instead of a subquery
- a
leftjoin to the same subquery - a
whereon the primary alias
- The result is the same with
autoIndex: 'off'and withautoIndex: 'eager'+BasicIndex.
| join | source | autoIndex | result |
|---|---|---|---|
| inner | collection | off / eager | ✅ ["p2"] |
| inner | subquery | off / eager | ❌ ["p1","p2"] |
| left | collection | off / eager | ✅ ["p2"] |
| left | subquery | off / eager | ✅ ["p2"] |
To Reproduce
npm i @tanstack/db@0.9.2, then node repro.mjs:
// repro.mjs
import {
BasicIndex,
createCollection,
createLiveQueryCollection,
eq,
isNull,
localOnlyCollectionOptions,
Query,
} from '@tanstack/db'
function collections(autoIndex) {
const index = autoIndex === 'eager' ? { autoIndex, defaultIndexType: BasicIndex } : { autoIndex }
const posts = createCollection(
localOnlyCollectionOptions({
getKey: (r) => r.id,
...index,
initialData: [
{ id: 'p1', authorId: 'a1' },
{ id: 'p2', authorId: 'a2' },
],
}),
)
const authors = createCollection(
localOnlyCollectionOptions({
getKey: (r) => r.id,
...index,
initialData: [
{ id: 'a1', name: 'Alice', deletedAt: null },
{ id: 'a2', name: 'Bob', deletedAt: null },
],
}),
)
return { posts, authors }
}
async function run(label, { autoIndex, type, source }) {
const { posts, authors } = collections(autoIndex)
const joined =
source === 'subquery'
? new Query().from({ author: authors }).where(({ author }) => isNull(author.deletedAt))
: authors
const q = createLiveQueryCollection((q) =>
q
.from({ post: posts })
.join({ author: joined }, ({ post, author }) => eq(author.id, post.authorId), type)
// Outer where on the joined alias: only Bob's post should remain.
.where(({ author }) => eq(author.name, 'Bob'))
.select(({ post }) => ({ id: post.id })),
)
await q.preload()
const ids = q.toArray.map((r) => r.id).sort()
const ok = JSON.stringify(ids) === JSON.stringify(['p2'])
console.log(`${ok ? 'ok ' : 'FAIL'} ${label.padEnd(44)} -> ${JSON.stringify(ids)}`)
}
for (const autoIndex of ['off', 'eager']) {
for (const type of ['inner', 'left']) {
for (const source of ['collection', 'subquery']) {
await run(`${type} join, ${source} source, autoIndex ${autoIndex}`, { autoIndex, type, source })
}
}
}
Output (the "Join requires an index" warnings for the autoIndex: 'off' runs are omitted):
ok inner join, collection source, autoIndex off -> ["p2"]
FAIL inner join, subquery source, autoIndex off -> ["p1","p2"]
ok left join, collection source, autoIndex off -> ["p2"]
ok left join, subquery source, autoIndex off -> ["p2"]
ok inner join, collection source, autoIndex eager -> ["p2"]
FAIL inner join, subquery source, autoIndex eager -> ["p1","p2"]
ok left join, collection source, autoIndex eager -> ["p2"]
ok left join, subquery source, autoIndex eager -> ["p2"]
The .innerJoin() shorthand behaves the same way. In the variant below Bob is soft-deleted, so the subquery excludes him. The outer where asks for Bob, so the result should be empty, but Alice's post comes back: the subquery's where was applied and the outer one was not.
const sub = new Query().from({ author: authors }).where(({ author }) => isNull(author.deletedAt))
q.from({ post: posts })
.innerJoin({ author: sub }, ({ post, author }) => eq(author.id, post.authorId))
.where(({ author }) => eq(author.name, 'Bob')) // Bob has deletedAt set
// expected [], got ["p1"]
Expected behavior
A where on an inner-joined alias narrows the result the same way whether the join source is a collection or a subquery. Here that means ["p2"], as the other seven combinations return.
Screenshots
N/A. The script output above shows it.
Desktop (please complete the following information):
- OS: macOS 27.0
- Runtime: Node.js v24.16.0 (no browser needed)
- Version:
@tanstack/db@0.9.2
Additional context
We first saw this in a React app (@tanstack/react-db + query-db-collection): a table filter on a joined row "didn't apply until a page reload". The join to the joined table was a subquery because it filters out soft-deleted rows. After a reload the collections held only the server's already-filtered rows, which hid the bug.
Our workaround: for inner joins, join the bare collection and apply the subquery's conditions as an outer where. For an inner join that's equivalent, and it filters correctly.
The only other issues I found about subquery join sources don't match this shape. #1590 (left join onto a subquery that itself contains a join) and #1709 (join inside a correlated include) are both closed.
- Ngôn ngữ chính
- TypeScript
- Star
- 3.9k
- Fork
- 266
- Merge trung bình
- 1 ngày 17 giờ
- Pull request đã merge (30 ngày)
- 63
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của TanStack/db
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
Index suggestion for collection size is gated on autoIndex, so it only fires where it is redundant Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
useLiveSuspenseQuery never releases over an on-demand collection whose loadSubset returns a Promise Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 55/100
Issue tương tự
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Crush Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
ElementsProject/cln-application#167 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Quantco/pnpm-licenses#17 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100