Problem

Nested SAMPLE BY queries with an ORDER BY clause in between incorrectly failed with:

ASC order over TIMESTAMP column is required but not provided

Example of failing query:

SELECT timestamp, count()
FROM (
    (
        SELECT timestamp, symbol, count(bids[1][1]) as total
        FROM eq_equities_market_data
        WHERE symbol = 'HSBC'
        SAMPLE BY 10s
    )
    ORDER BY timestamp
)
SAMPLE BY 10m

Root Cause

When SAMPLE BY needs ordered timestamp data, it pushes timestampRequired=true onto a context stack. This flag propagates to nested models and throws an error if ASC timestamp order isn't guaranteed.

However, when there's an ORDER BY clause between the outer and inner SAMPLE BY, the inner model's ordering is irrelevant since ORDER BY will re-sort the data anyway.

Fix

When an ORDER BY clause is present, push timestampRequired=false to signal that nested models don't need to maintain timestamp ordering.

🤖 Generated with Claude Code