feat: mentions handling on tinybird#3614
Conversation
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
2 similar comments
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
This is the final PR Bugbot will review for you during this billing cycle
Your free Bugbot reviews will reset on December 12
Details
Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
1 similar comment
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
| `language` String `json:$.language` DEFAULT '', | ||
| `projectSlug` LowCardinality(String) `json:$.projectSlug` DEFAULT '', | ||
| `createdAt` DateTime64(3) `json:$.createdAt` DEFAULT now64(3), | ||
| `bookmarked` UInt8 `json:$.bookmarked`, |
There was a problem hiding this comment.
Bug: Missing DEFAULT value for bookmarked field
The bookmarked field is defined as UInt8 without a DEFAULT value. All other UInt8 fields in the codebase consistently have DEFAULT 0 or DEFAULT 1. Without a default, the field becomes required in incoming JSON, which will cause insertion failures when the field is missing from webhook payloads.
| `projectSlug` LowCardinality(String) `json:$.projectSlug` DEFAULT '', | ||
| `createdAt` DateTime64(3) `json:$.createdAt` DEFAULT now64(3), | ||
| `bookmarked` UInt8 `json:$.bookmarked`, | ||
| `keywords` Array(String) `json:$.keywords[:]` |
There was a problem hiding this comment.
Bug: Missing DEFAULT value for keywords array
The keywords field is defined as Array(String) without a DEFAULT value. All other Array(String) fields in the codebase consistently have DEFAULT []. Without a default, the field becomes required in incoming JSON, which will cause insertion failures when the field is missing from webhook payloads.
Signed-off-by: Gašper Grom <gasper.grom@gmail.com>
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
|
Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability. Example:
Projects:
Please add a Jira issue key to your PR title. |
2 participants
Note
Introduces a new mentions datasource, integrates aggregated community fields into insights projects, adds a mentions listing pipe, and standardizes Kafka sink configs.
mentions: New datasource capturing community mentions with sentiment, relevance, and metadata.insights_projects_populated_dswithcommunityPlatforms,communityKeywords,communityLanguages.insights_projects_populated_copy.pipe): Aggregate mentions perprojectSlugand LEFT JOIN into results.insightsProjects_filtered.pipe): Select and return new community fields.mentions_list.pipe: List mentions with filters forprojectSlug,platforms,keywords,sentiments,languages, with pagination.health_score_sink.pipe,insights_projects_populated_sink.pipe: NormalizeTYPE SINK; addEXPORT_FORMAT csvandEXPORT_STRATEGY @new; keep Kafka topic and schedule.Written by Cursor Bugbot for commit ed79abe. This will update automatically on new commits. Configure here.