Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

doc: JSON_OBJECTAGG requires MariaDB >= 10.5 #287

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

oxzi
Copy link
Member

@oxzi oxzi commented Mar 10, 2025

There was a bug report in the Icinga Community Forum1 regarding an unknown SQL function when using MariaDB 10.3.39. The function in question was JSON_OBJECTAGG, only being used in one place by web2.

MariaDB's documentation states that this function was introduced in version 10.5.03, greater than the currently documented infimum. This fixes our docs.

This function - or PostgreSQL's used counterpart json_object_agg - is available in the required versions for MySQL and PostgreSQL4 5.

Footnotes

  1. https://community.icinga.com/t/current-state-of-icinga-notifications-modules/14666

  2. https://github.com/Icinga/icinga-notifications-web/blob/ede7734436ddee952f89295932fbc05e0b4af6e7/library/Notifications/Model/Behavior/IdTagAggregator.php#L58-L59

  3. https://mariadb.com/kb/en/json_objectagg/

  4. https://dev.mysql.com/doc/refman/5.7/en/aggregate-functions.html#function_json-objectagg

  5. https://www.postgresql.org/docs/9.6/functions-aggregate.html

There was a bug report in the Icinga Community Forum[0] regarding an
unknown SQL function when using MariaDB 10.3.39. The function in
question was JSON_OBJECTAGG, only being used in one place by web[1].

MariaDB's documentation states that this function was introduced in
version 10.5.0[2], greater than the currently documented infimum. This
fixes our docs.

This function - or PostgreSQL's used counterpart json_object_agg - is
available in the required versions for MySQL and PostgreSQL[3][4].

[0]: https://community.icinga.com/t/current-state-of-icinga-notifications-modules/14666
[1]: https://github.com/Icinga/icinga-notifications-web/blob/ede7734436ddee952f89295932fbc05e0b4af6e7/library/Notifications/Model/Behavior/IdTagAggregator.php#L58-L59
[2]: https://mariadb.com/kb/en/json_objectagg/
[3]: https://dev.mysql.com/doc/refman/5.7/en/aggregate-functions.html#function_json-objectagg
[4]: https://www.postgresql.org/docs/9.6/functions-aggregate.html
@oxzi oxzi added the doc Documentation related label Mar 10, 2025
@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed CLA is signed by all contributors of a PR doc Documentation related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant