Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
44 changes: 0 additions & 44 deletions .github/workflows/analytics-engine-compat.yml

This file was deleted.

820 changes: 274 additions & 546 deletions .github/workflows/ppl-lint-multiversion-validation.yml

Large diffs are not rendered by default.

16 changes: 8 additions & 8 deletions docs/dev/ppl-lint-analytics-engine-ci-validation.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,13 +89,13 @@ The repository already contains most of the required test infrastructure:
`LogicalTableScan(table=[[opensearch,` and not
`CalciteLogicalIndexScan`.

### 2.3 Gap in the existing analytics workflow
### 2.3 Gap in the retired analytics workflow

`.github/workflows/analytics-engine-compat.yml` runs only
`AnalyticsEngineCompatIT`. Its purpose is plugin coexistence. Its PPL assertion
uses the `rest` row source, which is explicitly excluded from analytics
routing. The workflow can therefore pass without executing a PPL query through
DataFusion.
The removed `.github/workflows/analytics-engine-compat.yml` ran only
`AnalyticsEngineCompatIT`. Its purpose was plugin coexistence. Its PPL
assertion used the `rest` row source, which is explicitly excluded from
analytics routing. The workflow could therefore pass without executing a PPL
query through DataFusion.

The `analyticsEngineCompat` cluster is also intentionally smaller than the
stack required for real analytics execution. It does not install the composite
Expand Down Expand Up @@ -763,8 +763,8 @@ After the required lane is stable, evaluate:
- Matching released analytics stacks.
- A scheduled three-shard analytics leg.
- Analytics execution for the discovery corpus.
- Consolidating or retiring redundant parts of
`analytics-engine-compat.yml`.
- Reintroducing a standalone analytics workflow only if it validates the
routed analytics contract described here.

These are separate changes and are not prerequisites for initial enforcement.

Expand Down
208 changes: 87 additions & 121 deletions scripts/ppl-lint/README.md

Large diffs are not rendered by default.

Loading
Loading