[pull] master from cube-js:master#566
Merged
Merged
Conversation
…ia match-only mode (#11178) The pre-aggregation refresh/metadata path (e.g. the `/cubejs-system/v1/pre-aggregations` API) builds a query from a rolling pre-agg's references (rolling measure + a time dimension with granularity, but no date range) and calls `CompilerApi.getSql()`, which 500s. `getSql()` eagerly builds both `.sql` (`buildSqlAndParams`, discarded by the refresh path) and `.preAggregations`; on dialects without generated time series the native matching itself also throws, because `findPreAggregationForQueryRust` builds the full outer SQL — including the rolling-window time series, which requires a date range — just to compute the match.
#11179) A measure referenced only in ORDER BY (not in the selected measures) cannot be served from a pre-aggregation read: CubeStore's DataFusion rejects an ORDER BY that aggregates a rollup column which isn't in the SELECT projection, failing with e.g. `No field named e_commerce___t_a_external.ec__total_profit`.
#11181) When an ungrouped query matched a pre-aggregation, the measure was rendered through the ungrouped measure nodes without the pre-aggregation reference substitution, so it emitted the measure's base-table SQL (e.g. the raw `amount` column) instead of the stored rollup column. Reading the rollup then failed with a schema error like `No field named big_e_commerce.sales`. Wrap the ungrouped measure nodes with the pre-aggregation reference map when reading from a rollup. Unlike the grouped case the stored column already holds the aggregated value and is returned as-is, so it is a plain column reference with no `sum()` wrap.
#11185) * docs(embedding): document iframe events & actions postMessage contract Add a new "Events & actions" page under Iframe embedding documenting the embed↔host postMessage contract: the message envelope, the 8 outbound events (ready, view, navigate, dashboard-loaded, download, drilldown, ai-query, error) with payloads and examples, the 6 inbound actions (set-color-scheme, set-theme, set-locale, set-filter, navigate, refresh), surfaces, origin-validation security guidance, and a complete vanilla-JS host example. No SDK required — plain window.postMessage. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * docs(embedding): format event payload examples one field per line Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * docs(embedding): address review — Note callout, surface clarity, embedTheme naming Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Add a Localization page covering the supported languages, the account-wide default (Embed → Settings → Language), the per-embed ?locale= URL parameter, and the runtime cube:action:set-locale override, with the precedence order between them. Cross-link from the dashboards, events, and customization pages and register the new page in docs.json. Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )