Skip to content

Conversation

rakhi-prathap
Copy link
Contributor

@rakhi-prathap rakhi-prathap commented Mar 26, 2025

Copy link

@dhanyagowrish dhanyagowrish left a comment

Choose a reason for hiding this comment

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

The description of the DOC ticket (DOC-10857) includes the reason why ARRAY_REMOVE() cannot remove NULL values from an array. Can that be added?

Reasoning from the DOC ticket:

The ARRAY_REMOVE function can't be used to remove NULL values from an array since it uses an equality predicate and NULL doesn't equal NULL.

The internal implementation of the ARRAY_REMOVE() function uses an equality predicate when evaluating the array elements to remove. And since NULL does not equal NULL, the implementation will not remove the NULL values from the array,

@rakhi-prathap
Copy link
Contributor Author

rakhi-prathap commented Apr 23, 2025

The description of the DOC ticket (DOC-10857) includes the reason why ARRAY_REMOVE() cannot remove NULL values from an array. Can that be added?

Reasoning from the DOC ticket:

The ARRAY_REMOVE function can't be used to remove NULL values from an array since it uses an equality predicate and NULL doesn't equal NULL.

The internal implementation of the ARRAY_REMOVE() function uses an equality predicate when evaluating the array elements to remove. And since NULL does not equal NULL, the implementation will not remove the NULL values from the array,

Done. Updated the doc to include this information.

Copy link
Contributor

@RichardSmedley RichardSmedley left a comment

Choose a reason for hiding this comment

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

👍

@rakhi-prathap rakhi-prathap merged commit c80cf2e into release/7.6 Apr 24, 2025
6 checks passed
@rakhi-prathap rakhi-prathap deleted the DOC-10857-update-array-remove branch April 24, 2025 04:45
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