Uploaded image for project: 'MariaDB Server'
  1. MariaDB Server
  2. MDEV-32854

Make JSON_DEPTH_LIMIT configurable

Details

    • Q2/2025 Development

    Description

      The current JSON_DEPTH_LIMIT is 32 as defined in include/json_lib.h
      Source code:
      https://github.com/MariaDB/server/blob/22ffe4dd227451ebd0ef842850ef57b81aa8f930/include/json_lib.h#L10

      This limit is not documented.

      Customer requests to make this limit configurable with a global variable.

      Attachments

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. MCOL-5885 Adopt the recent MDB's JSON parser lib dynamic array root mem changes

          • Major - Major loss of function.
          • Open

          Task - A task that needs to be done. MDEV-26877 Deeply nested vaid JSON values seem invalid ( JSON_DEPTH_LIMIT )

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Bug - A problem which impairs or prevents the functions of the product. MDEV-30343 Optimizer trace produces JSON too deep for MariaDB JSON functions

          • Major - Major loss of function.
          • Confirmed

          Bug - A problem which impairs or prevents the functions of the product. MDEV-33437 JSON_SEARCH fails to find match

          • Major - Major loss of function.
          • Closed

          Activity

            Ascending order - Click to sort in descending order
            View 24 older comments
            akim.demaille Akim Demaille added a comment -

            Hi,
            So what did the efficiency testing give?
            The first implementations were submitted one year ago I have no doubt you guys have lots of things to do, but this is serious limitation.
            Cheers!

            akim.demaille Akim Demaille added a comment - Hi, So what did the efficiency testing give? The first implementations were submitted one year ago I have no doubt you guys have lots of things to do, but this is serious limitation. Cheers!
            ramesh Ramesh Sivaraman added a comment - - edited

            ok to push

            ramesh Ramesh Sivaraman added a comment - - edited ok to push
            akim.demaille Akim Demaille added a comment -

            Hi,
            Does this mean that this time it's going to be released for sure?
            This bug cripples the JSON support of Maria.

            akim.demaille Akim Demaille added a comment - Hi, Does this mean that this time it's going to be released for sure? This bug cripples the JSON support of Maria.
            sanja Oleksandr Byelkin added a comment -

            akim.demaille if it pass internal tests, so it does not (performance degradation)

            sanja Oleksandr Byelkin added a comment - akim.demaille if it pass internal tests, so it does not (performance degradation)
            rucha174 Rucha Deodhar added a comment -

            Hi, edward, I am really hoping for it to go in upcoming release. There is some performance related problem that needs to be fixed (in some cases a lot of performance degradation for some json functions) + changes to be made in columnstore. That is all there is there is left to do.

            rucha174 Rucha Deodhar added a comment - Hi, edward , I am really hoping for it to go in upcoming release. There is some performance related problem that needs to be fixed (in some cases a lot of performance degradation for some json functions) + changes to be made in columnstore. That is all there is there is left to do.

            People

              rucha174 Rucha Deodhar
              edward Edward Stoever
              Votes:
              6 Vote for this issue
              Watchers:
              15 Start watching this issue

              Dates

                Created:
                Updated:

                Git Integration

                  Error rendering 'com.xiplink.jira.git.jira_git_plugin:git-issue-webpanel'. Please contact your Jira administrators.