Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Move remaining citus_internal functions #7478

Merged
merged 9 commits into from
Feb 7, 2024
Merged

Conversation

eaydingol
Copy link
Contributor

@eaydingol eaydingol commented Feb 2, 2024

Moves the following functions to the Citus internal schema:

citus_internal_local_blocked_processes
citus_internal_global_blocked_processes
citus_internal_mark_node_not_synced
citus_internal_unregister_tenant_schema_globally
citus_internal_update_none_dist_table_metadata
citus_internal_update_placement_metadata
citus_internal_update_relation_colocation
citus_internal_start_replication_origin_tracking
citus_internal_stop_replication_origin_tracking
citus_internal_is_replication_origin_tracking_active

#7405

…_blocked_processes. Update the udfs referencing to them.
…r_tenant_schema_globally, update_placement_metadata.
@eaydingol eaydingol marked this pull request as draft February 2, 2024 07:53
Copy link

codecov bot commented Feb 2, 2024

Codecov Report

Merging #7478 (63ccdac) into main (6869b3a) will decrease coverage by 0.31%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7478      +/-   ##
==========================================
- Coverage   89.59%   89.28%   -0.31%     
==========================================
  Files         282      282              
  Lines       60310    60312       +2     
  Branches     7512     7512              
==========================================
- Hits        54033    53848     -185     
- Misses       4121     4265     +144     
- Partials     2156     2199      +43     

…top_replication_origin_tracking, is_replication_origin_tracking_active to citus_internal schema
@eaydingol eaydingol requested a review from JelteF February 2, 2024 11:54
@eaydingol eaydingol marked this pull request as ready for review February 2, 2024 11:54
eaydingol and others added 3 commits February 7, 2024 16:34
Co-authored-by: Jelte Fennema-Nio <jelte.fennema@microsoft.com>
Co-authored-by: Jelte Fennema-Nio <jelte.fennema@microsoft.com>
@eaydingol eaydingol merged commit f01c5f2 into main Feb 7, 2024
157 checks passed
@eaydingol eaydingol deleted the move_remaining_functions2 branch February 7, 2024 13:58
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants