Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pre-agg combination of config not sufficiently explained #7532

Closed
jlloyd-widen opened this issue Dec 14, 2023 · 1 comment
Closed

Pre-agg combination of config not sufficiently explained #7532

jlloyd-widen opened this issue Dec 14, 2023 · 1 comment
Assignees
@igorlukanin
Labels
docs Issues that require a documentation improvement

Comments

@jlloyd-widen
Copy link
Contributor

cube/docs/pages/reference/data-model/pre-aggregations.mdx

Line 961 in cc4092e

Partition tables are refreshed as a whole. When a new partition table is

In this section of the docs, it would helpful if it stated that you cannot use the incremental key when you are using the sql key.

For that matter, it would significantly help this particular page if some regular attention was made to what combinations of config can/should be used together and what cannot as it related, at least, to pre-aggs.
@igorlukanin igorlukanin added the docs Issues that require a documentation improvement label Jan 3, 2024
@igorlukanin igorlukanin self-assigned this Jan 3, 2024
@igorlukanin
Copy link
Member

@jlloyd-widen Great suggestion, thank you!

igorlukanin added a commit that referenced this issue Jul 10, 2024
@igorlukanin
docs: Misc fixes. Closes #7286 and #7532
5bd5270
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@igorlukanin igorlukanin

Labels
docs Issues that require a documentation improvement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@igorlukanin @jlloyd-widen