Skip to content

fix: add native sum column analyzer - #1418

Merged
czgu merged 1 commit into
pinterest:masterfrom
rchandnaeg:fix/support-sum-column-analyzer
Mar 1, 2024
Merged

czgu merged 1 commit into
pinterest:masterfrom
rchandnaeg:fix/support-sum-column-analyzer

Conversation

@rchandnaeg

Copy link
Copy Markdown
Contributor

I know that there's the option to add custom column analyzers, but I think sum is common enough to be supported natively and appear at the top of the Quick Insights view of a column.

Screenshot 2024-03-01 at 11 42 57 AM

@rchandnaeg rchandnaeg changed the title fix: add support for a sum column analyzer fix: add native sum column analyzer Mar 1, 2024

@czgu czgu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you! great idea

@czgu
czgu merged commit e313560 into pinterest:master Mar 1, 2024
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

Successfully merging this pull request may close these issues.

2 participants