Skip to content

Commit edce116

Browse files
rustyconoverclaude
andcommitted
ci: HTTP revalidation now implemented; skip stays until releases ship
Update the revalidate.test http-lane skip comment: HTTP conditional revalidation is now implemented (C++ /init-request validators + vgi-rpc surfacing them to the producer's first process()), but this lane runs the published community vgi extension + PyPI vgi-rpc, so keep the skip until both ship a release carrying the change, then remove the line. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 6e0301a commit edce116

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

‎ci/run-integration.sh‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,12 +73,12 @@ if [ "$TRANSPORT" = "http" ]; then
7373
# * buffer_input/sizes.test — input buffering semantics differ over HTTP
7474
# (same known limitation). (scale.test_slow is a .test_slow file and is
7575
# never staged below, which only finds *.test.)
76-
# * cache/revalidate.test — conditional revalidation (304) is subprocess-only:
77-
# the if_none_match validator rides the first producer tick, but over HTTP
78-
# that first exchange folds into the /init POST before any tick, so the
79-
# not_modified path can't fire (documented in the test header + vgi CLAUDE.md).
80-
# The test omits `require httpfs` to auto-skip under a plain HTTP harness, but
81-
# this lane force-injects `LOAD httpfs`, defeating that — so drop it explicitly.
76+
# * cache/revalidate.test — HTTP conditional revalidation (304) IS implemented
77+
# (the C++ client puts the validators on the /init request metadata and vgi-rpc
78+
# surfaces them to the producer's first process()), but it needs BOTH a
79+
# community vgi extension carrying that C++ change AND a vgi-rpc release carrying
80+
# the _app_stream change. This lane runs the published community extension +
81+
# PyPI vgi-rpc, so keep it skipped until both ship, then remove this line.
8282
EXTRA_SKIP=(
8383
-not -name 'projection_pushdown_repro.test'
8484
-not -name 'dynamic_filter.test'

0 commit comments

Comments
 (0)