Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-58652

Allow large files in QJson parser

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P2: Important
    • 5.15
    • 5.8.0
    • Core: QString and Unicode
    • None
    • Windows

    Description

      Hello everyone!

      I came across the error message "document too large" when I tried to parse a 135 MB JSON file. It seems there is an arbitrary limit of Value::MaxSize although I have enough RAM memory available on the machine.

      I found out this limit was introduced with this bug fix: QTBUG-29288

      However, I wonder if the bug with too large files couldn't be fixed without the fixed limit. I took a look at the source code, but as I do not have the full insight in all the dependencies of such low-level functionality, I want to ask you experts for help/advise/fix.

      Thank you and best regards,
      Daniel

      Attachments

        1. json_20180507.patch
          15 kB
          Dmitriy Purgin

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-60728 QJsonObject repetitive read/write of a key (large) results in error

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Reported
          resulted from

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-29288 Qt5 QJsonDocument method 'toJson()' will crash with bigger arrays

          • P1: Critical - Urgent and Important, will STOP the release if matched with set FixVersion field. This includes regressions from the last version that are not edge cases; Data loss; Build issues; All but the most unlikely crashes/lockups/hanging; Serious usability issues and embarrassing bugs & Bugs critical to a deliverable. This is the highest possible priority for requirements.
          • Closed
          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              laknoll Lars Knoll
              dhoegerl Daniel Hoegerl
              Votes:
              9 Vote for this issue
              Watchers:
              15 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes