Skip to content

feat(db-ops): Allow schemas to have ownership set at creation [ENG_OPER-10194]#4

Closed
bricklenclickup wants to merge 3 commits intoclickup:mainfrom
time-loop:feature/ENG_OPER-10194_schema_ownership
Closed

feat(db-ops): Allow schemas to have ownership set at creation [ENG_OPER-10194]#4
bricklenclickup wants to merge 3 commits intoclickup:mainfrom
time-loop:feature/ENG_OPER-10194_schema_ownership

Conversation

@bricklenclickup
Copy link

Allow microsharding schemas to be created with a specific owner.
https://app.clickup-stg.com/t/333/ENG_OPER-10194

@@ -1,6 +1,9 @@
DROP FUNCTION IF EXISTS microsharding_ensure_exist(integer, integer);
Copy link
Author

Choose a reason for hiding this comment

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

Was going to add the entire file in a transaction but decided against it.
The DROP FUNCTION above is to ensure the old function is removed if it exists.

COMMENT ON FUNCTION microsharding_ensure_exist(integer, integer)
IS 'Creates shards (schemas) in the range shard_from..shard_to (inclusive).';
COMMENT ON FUNCTION microsharding_ensure_exist(integer, integer, varchar)
IS 'Creates shards (schemas) in the range shard_from..shard_to (inclusive).'; No newline at end of file

Choose a reason for hiding this comment

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

sorry nit: can we have new line here
CleanShot 2025-10-14 at 16 34 54@2x

Copy link

@elliotvilhelm elliotvilhelm left a comment

Choose a reason for hiding this comment

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

cc: @dimikot

@bricklenclickup
Copy link
Author

Closing in favour of time-loop#1

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.

3 participants