Repository navigation
board: reconciliation after #1674–#1682 — D-QSP-0 row, D-RFT-1 row, polyfill map G1, PR_ARC - #1683
Conversation
…1680/#1682, boundaries Only this session's rows and entry: the STATUS_BOARD "D-QSP-0 remainder" row (In PR -> Shipped, slice 2 added), PR_ARC entries for #1675, #1680 and #1682 prepended, and the entry now states that Quack has no NULL-literal Filter variant, that the PostgreSQL examples are lab evidence and not CI, and #1682's final CI (18/18 green on 96e8010). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
…correction #1686 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA
This PR is board-only. It is the board part (E) of the consolidated review
GPT:gpt-ada-01/20261021-pr1674-1682-consolidated-review-01. I first claimed it in EXO-50/036; on the operator's instruction it was widened in/038to cover rows belonging to other sessions.Changes
STATUS_BOARD.mda55fc67), board: correct the D-QSP-0 entry's G1 claim (NULL literals need no new carrier) #1680 (b306516) and quack: pin SQL NULL operands over existing sql_where (tests + PostgreSQL differential) #1682 (da9b755), and gives an explicit list of what is not shipped:COALESCE/NULLIF;NOT IN.0f756b98). The row now names the open correction r2il: bool_or/bool_and keep NULL apart from FALSE; COUNT(DISTINCT) excludes NULL keys (D-RFT-1 correction) #1686: on main,bool_or/bool_andread NULL as FALSE, andcount_distinct_sortedcounts NULL keys.PR_ARC_INVENTORY.md: entries for #1675, #1680 and #1682 are prepended, with 21 lines added and none removed.entries/2026-10-21-pg-classview-polyfill-map.md(file owned by session 01G22y; edited on the operator's instruction): the three G1 rows and gap item 1 are marked ⊘ corrected, each naming the old text it replaces.Filtervariant, and the tested subset needs none: a NULL operand is a lane alias with an all-clear validity plane undersql_where(quack: pin SQL NULL operands over existing sql_where (tests + PostgreSQL differential) #1682).IS NOT DISTINCT FROM NULLisIS [NOT] NULL; it is composable but has no named builder.NOT IN, and nullable arithmetic.entries/2026-10-21-d-qsp-0-scalar-aggregates-null.md: a Boundaries paragraph states that:96e8010.Checks
Related correction PRs
WhereSetand GKAT duality: contract: normalize_kleene returns WhereSet (WHERE-terminal only); GKAT Where/Check negation duality pinned #1687.The entries edited by those PRs are their own.
I will not merge this PR; it waits for your review.
🤖 Generated with Claude Code
https://claude.ai/code/session_013z2e1YXkm6o6HNrDn5BPWA