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

Migrate stats queries to new schema #62

Open
mmerdes opened this issue Feb 14, 2025 · 1 comment
Open

Migrate stats queries to new schema #62

mmerdes opened this issue Feb 14, 2025 · 1 comment
Assignees
@mcauer @mmerdes

Comments

@mmerdes
Copy link
Collaborator

mmerdes commented Feb 14, 2025 •
edited
Loading

Currently, the main table stats_2 has unnested hashtags.
The new schema has hashtags as array.

The goal of this issue is to reproduce all endpoints based on new table all_stats_3
without adding new functionality.
mmerdes added a commit that referenced this issue Feb 14, 2025
@mmerdes @mcauer
test: add assertions for schema version to topic test #62
baa29d9 
'Migrate stats queries to new schema'
#62

Co-authored-by: Michael Auer <[email protected]>
mmerdes added a commit that referenced this issue Feb 17, 2025
@mmerdes @mcauer
feature: make stats and topic schema version configurable separately,…
9d1c8db 
… use version 3 for stats #62

'Migrate stats queries to new schema'
#62

Co-authored-by: Michael Auer <[email protected]>
mmerdes added a commit that referenced this issue Feb 17, 2025
@mmerdes
feature: make stats and topic schema version configurable separately …
e1b624e 
…- cleanup impl #62

'Migrate stats queries to new schema'
#62
mmerdes added a commit that referenced this issue Feb 17, 2025
@mmerdes
fix: use correct parameterization of sql generator #62
4cc25c2 
'Migrate stats queries to new schema'
#62
mmerdes added a commit that referenced this issue Feb 17, 2025
@mmerdes
feature: adapt stats queries to use array of hashtags instead of scal…
3ec9914 
…ar hashtag #62

'Migrate stats queries to new schema'
#62
mmerdes added a commit that referenced this issue Feb 17, 2025
@mmerdes
fix: fix detekt quality check by allowing a longer parameter list in …
86571bb 
…2 cases #62

'Migrate stats queries to new schema'
#62
mmerdes added a commit that referenced this issue Feb 17, 2025
@mmerdes
refactor: use has_hashtags flag in schema and test data - TODO: use…
bd39dc8 
… in some queries #62

'Migrate stats queries to new schema'
#62
mmerdes added a commit that referenced this issue Feb 17, 2025
@mmerdes
fix: use fixed version 24.9.2.42 for clickhouse in test-containers to…
a6533d4 
… fix build on nix-systems #62

'Migrate stats queries to new schema'
#62
mmerdes added a commit that referenced this issue Feb 18, 2025
@mmerdes
refactor: rename stats table from 'stats_2' to 'all_stats_3' #62
d794253 
'Migrate stats queries to new schema'
#62
@mmerdes
Copy link
Collaborator Author

mmerdes commented Feb 18, 2025

@mcauer
Let's review and improve this soon.

mmerdes added a commit that referenced this issue Feb 19, 2025
@mmerdes @mcauer
doc: remove obsolete todos #62
5bea747 
'Migrate stats queries to new schema'
#62

Co-authored-by: Michael Auer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mcauer mcauer

@mmerdes mmerdes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mcauer @mmerdes