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

useview_telemetry fails when app version is not available on telemetry record. #125

Open
1yuv opened this issue Nov 11, 2022 · 1 comment

Comments

@1yuv
Copy link
Member

1yuv commented Nov 11, 2022

Materialized view doesn't get created or fails if app version is not available in telemetry data.

useview_telemetry creates date based on the app version to handle different formatting used before 3.8 and later. However, if a telemetry log doesn't have app version, then date can not be created and it fails.

One of the incident we noticed was here. This one was fixed by adding following condition here

 AND doc#>>'{metadata,versions,app}' IS NOT NULL;

We may need to update default query with similar condition or handle in some other ways.

@derickl
Copy link
Member

derickl commented Apr 29, 2023

You could potentially also resolve this by adding the null check in the case...when block

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants