external
/
citus
mirror of https://github.com/citusdata/citus.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix whitespace

deprecation-mechanism
Alper Kocatas 2025-04-02 16:33:37 +00:00
parent b08c46ba08
commit 36fb5e7c0f
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
CONTRIBUTING.md
View File

@ -248,21 +248,21 @@ Any other SQL you can put directly in the main sql file, e.g.
8. After the tests pass on CI, fast-forward the release branch `git push origin release-11.3-<yourname>:release-11.3` 8. After the tests pass on CI, fast-forward the release branch `git push origin release-11.3-<yourname>:release-11.3`
### Deprecating features ### Deprecating features
Udf's and other API functions can be deprecated using the following mechanism: Udf's and other API functions can be deprecated using the following mechanism:
#### 1. Update the docs and mark the related function as "deprecated" #### 1. Update the docs and mark the related function as "deprecated"
Update the related documentation about the function/udf and indicate that the function has been deprecated. Update the related documentation about the function/udf and indicate that the function has been deprecated.
Additionally, add the name of the function to the "Deprecated features" section of release notes. Additionally, add the name of the function to the "Deprecated features" section of release notes.
#### 2. Mark the function as deprecated and emit a warning from the implementation #### 2. Mark the function as deprecated and emit a warning from the implementation
Use `ereport(WARNING, ...)` to raise a warning when the function is used. Additionally, add the keyword `<deprecated>` to the related comments. Use `ereport(WARNING, ...)` to raise a warning when the function is used. Additionally, add the keyword `<deprecated>` to the related comments.
Example: Example:
``` ```
/* /*
* <deprecated> * <deprecated>
* master_get_active_worker_nodes is a wrapper function for old UDF name. * master_get_active_worker_nodes is a wrapper function for old UDF name.
*/ */
Datum Datum
master_get_active_worker_nodes(PG_FUNCTION_ARGS) master_get_active_worker_nodes(PG_FUNCTION_ARGS)
{ {
if (SRF_IS_FIRSTCALL()) if (SRF_IS_FIRSTCALL())