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.

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

Reflections: dimensions_by_day not parsed correctly #94

Closed
ravjotbrar opened this issue Nov 25, 2022 · 2 comments · Fixed by #256
Closed

Reflections: dimensions_by_day not parsed correctly #94

ravjotbrar opened this issue Nov 25, 2022 · 2 comments · Fixed by #256
Labels
bug Something isn't working

Comments

@ravjotbrar
Copy link
Contributor

Looked further into the macro that should handle this logic and only case that’s handled is if dimensions_by_day is None.

Screen Shot 2022-11-24 at 8 16 49 PM

Looking at granularity of a datetime col after specifying it in dimensions and dimensions_by_day should change to 'Date' rather than Original.

@jlarue26 jlarue26 added the bug Something isn't working label Jan 29, 2023
@LuVlk
Copy link

LuVlk commented Nov 6, 2024

Although documentation says the macro argument is called dimensions_by_day (See: https://github.com/dremio/dbt-dremio/wiki/Using-Materializations-with-Dremio#reflections), the argument is (in version 1.7.0) actually called by_day_dimensions (See:

display=none, dimensions=none, by_day_dimensions=none, measures=none) %}
)

The example in the Wiki should be

{{ config(materialized='reflection', 
reflection_type='aggregate', 
dimensions=['Datetime0'], 
by_day_dimensions=['Datetime0'], 
measures=['Num3', 'Int2'], 
computations=['MIN,SUM','COUNT'], 
arrow_cache=true) }}

-- depends_on: {{ ref('my_anchor') }}

@bcmeireles
Copy link
Contributor

Approached in #256

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging a pull request may close this issue.

4 participants