Skip to content

feat: optimized event tags table writes#386

Open
Roiocam wants to merge 6 commits intoapache:mainfrom
Roiocam:optimized-event-tags
Open

feat: optimized event tags table writes#386
Roiocam wants to merge 6 commits intoapache:mainfrom
Roiocam:optimized-event-tags

Conversation

@Roiocam
Copy link
Copy Markdown
Member

@Roiocam Roiocam commented Nov 26, 2025

No description provided.

@Roiocam Roiocam force-pushed the optimized-event-tags branch from ae45757 to df40ca0 Compare November 26, 2025 16:22
@pjfanning pjfanning mentioned this pull request Nov 27, 2025
@pjfanning pjfanning force-pushed the optimized-event-tags branch from 0503de2 to 3378e14 Compare November 28, 2025 12:29
@pjfanning
Copy link
Copy Markdown
Member

pjfanning commented Nov 28, 2025

@Roiocam I took your fixes for the typos and applied then separately. I rebased your PR if that is ok.

I have a question about the migration sql files. They seem useful but would I be right in saying that users will need to run them themselves or are they automatically applied? The reason I ask is that I plan to push on the 1.2.0 release and there are db schema changes in it and I'm wondering if I should add migration scripts to 1.2.0. So far, I just document in the release notes that there are changes and that users can apply them for themselves. #390

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