chore: adjust primitive query actions to normalized tables#1075
Merged
MicBun merged 1 commit intofeat/normalized-tablesfrom Jul 17, 2025
Merged
chore: adjust primitive query actions to normalized tables#1075MicBun merged 1 commit intofeat/normalized-tablesfrom
MicBun merged 1 commit intofeat/normalized-tablesfrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Time Submission Status
|
MicBun
pushed a commit
that referenced
this pull request
Aug 12, 2025
* chore: add data providers table * feat: add data_providers table * chore: add unique index * chore: create index if not exists * chore: normalize streams table (#1046) * chore: normalize metadata table (#1047) * chore: normalize taxonomies table (#1053) * chore: normalize primitive events table (#1054) * chore: adjust insert actions to normalized tables * chore: revise error message * feat: create action for data provider and test * chore: adjust insert metadata * feat: adjust insert primitive events * chore: adjust tests * chore: adjust tn ops tests * chore: adjust tests * chore: optimize stream ids lookup * fix: variable name * chore: use inner join * chore: remove unnecessary select * chore: use optimized action on truflation query too * chore: adjust common actions to normalized tables (#1068) * chore: adjust common actions to normalized tables * fix: join data providers * chore: debug * chore: adjust authorization actions to normalized tables (#1070) * chore: adjust authorization actions to normalized tables * chore: update actions * chore: update actions * fix: logic * chore: adjust taxonomy actions to normalized tables (#1074) * chore: adjust taxonomy actions to normalized tables * chore: avoid rate limiting on github actions * chore: adjust primitive query actions to normalized tables (#1075) * chore: adjust composed query actions to normalized tables (#1076) * chore: adjust composed query actions to normalized tables * chore: update derivates * fix: stream ref for parent * chore: adjust utilities actions to normalized tables (#1078) * chore: adjust utilities actions to normalized tables * chore: reduce database size expectation on test * chore: update truflation query * chore: use stream ref * fix: uuid to int * chore: register data provider on cache height tracking test * chore: remove unused local deployer * chore: remove unused local deployer * feat: add migration scripts for normalization process (#1097) * feat: add migration scripts for normalization process - Introduced multiple SQL migration scripts to transition the database schema from UUID to INT types for IDs. - Implemented actions to handle both old and new schemas during the migration. - Added scripts to populate `data_provider_id` and `id` columns in the `streams` table. - Created necessary indexes and constraints to optimize performance and maintain data integrity. - Included a shell script for executing migrations and logging execution times. - Documented the migration process and performance recommendations in the README. * chore: remove optimized index for gap-filling queries in primitive_events table * refactor: streamline SQL migration script for foreign key constraints - Added unique index on streams.id to ensure data integrity before creating foreign key references. - Simplified the addition of foreign key constraints for metadata and taxonomies tables. - Removed commented-out code related to partial indexes for primitive_events, as it is not currently supported. * fix: taxonomy test --------- Co-authored-by: Raffael Campos <raffael.rcampos@gmail.com>
MicBun
added a commit
that referenced
this pull request
Aug 19, 2025
* chore: add data providers table * feat: add data_providers table * chore: add unique index * chore: create index if not exists * chore: normalize streams table (#1046) * chore: normalize metadata table (#1047) * chore: normalize taxonomies table (#1053) * chore: normalize primitive events table (#1054) * chore: adjust insert actions to normalized tables * chore: revise error message * feat: create action for data provider and test * chore: adjust insert metadata * feat: adjust insert primitive events * chore: adjust tests * chore: adjust tn ops tests * chore: adjust tests * chore: optimize stream ids lookup * fix: variable name * chore: use inner join * chore: remove unnecessary select * chore: use optimized action on truflation query too * chore: adjust common actions to normalized tables (#1068) * chore: adjust common actions to normalized tables * fix: join data providers * chore: debug * chore: adjust authorization actions to normalized tables (#1070) * chore: adjust authorization actions to normalized tables * chore: update actions * chore: update actions * fix: logic * chore: adjust taxonomy actions to normalized tables (#1074) * chore: adjust taxonomy actions to normalized tables * chore: avoid rate limiting on github actions * chore: adjust primitive query actions to normalized tables (#1075) * chore: adjust composed query actions to normalized tables (#1076) * chore: adjust composed query actions to normalized tables * chore: update derivates * fix: stream ref for parent * chore: adjust utilities actions to normalized tables (#1078) * chore: adjust utilities actions to normalized tables * chore: reduce database size expectation on test * chore: update truflation query * chore: use stream ref * fix: uuid to int * chore: register data provider on cache height tracking test * chore: remove unused local deployer * chore: remove unused local deployer * feat: add migration scripts for normalization process - Introduced multiple SQL migration scripts to transition the database schema from UUID to INT types for IDs. - Implemented actions to handle both old and new schemas during the migration. - Added scripts to populate `data_provider_id` and `id` columns in the `streams` table. - Created necessary indexes and constraints to optimize performance and maintain data integrity. - Included a shell script for executing migrations and logging execution times. - Documented the migration process and performance recommendations in the README. * chore: remove optimized index for gap-filling queries in primitive_events table * refactor: streamline SQL migration script for foreign key constraints - Added unique index on streams.id to ensure data integrity before creating foreign key references. - Simplified the addition of foreign key constraints for metadata and taxonomies tables. - Removed commented-out code related to partial indexes for primitive_events, as it is not currently supported. * feat: view list taxonomies by height (#1098) * feat: view list taxonomies by height * chore: apply @coderrabit suggestion * feat: register deployer as a data provider in benchmark setup - Added functionality to register the deployer as a data provider during the benchmark setup process. - Included error handling to wrap any issues encountered while creating the data provider. * chore: remove unused SQL migration scripts for weight and composed data retrieval - Deleted the SQL files `get-all-weights.sql` and `get-composed-data.sql` as they are no longer needed in the migration process. - These scripts were previously used for retrieving weights and composed data from taxonomies but have been deemed obsolete. * chore: remove obsolete SQL migration scripts for normalization process - Deleted multiple SQL migration scripts related to the normalization process, including those for structure migration, data population, and final indexing. - These scripts were previously used to transition the database schema and have been deemed unnecessary after the migration completion. - The removal helps to clean up the codebase and reduce clutter in the migration directory. * refactor: normalize SQL migration scripts and clean up legacy columns - Updated SQL migration scripts to normalize references and improve data integrity. - Removed denormalized columns from `metadata`, `taxonomies`, and `primitive_events` tables. - Adjusted primary keys and added necessary indexes to optimize performance. - Cleaned up the migration directory by removing obsolete scripts and ensuring a streamlined structure. * chore: remove obsolete test files for weight and category streams - Deleted the test files `get_all_weights_for_query_test.go` and `get_category_test.go` as they are no longer needed in the codebase. - These tests were previously used for validating weight calculations and category stream functionalities but have been deemed unnecessary after recent refactoring and updates. - The removal helps to clean up the test directory and reduce clutter in the codebase. * feat: add tests for primitive batch insert alignment and enhance complex composed tests - Introduced `TestPrimitiveBatchInsertAlignment` to ensure correct mapping of batch insertions across multiple streams, addressing a regression issue with stream alignment. - Added `testComplexComposedRecordTruflationVariant` to validate the consistency of composed records when using Truflation-prefixed actions, ensuring compatibility with frozen logic. - Updated existing tests to register data providers consistently across various test cases, improving setup reliability. --------- Co-authored-by: williamrusdyputra <williamrusdyputra@gmail.com> Co-authored-by: Michael Buntarman <michaelboentarman@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
resolves: #1072
How Has This Been Tested?
query_test.gopassed