Uploaded image for project: 'yangtools'
  1. yangtools
  2. YANGTOOLS-936

JsonParserStream should not use recursion

    XMLWordPrintable

Details

    • Improvement
    • Status: Confirmed
    • Medium
    • Resolution: Unresolved
    • None
    • 14.0.0
    • codecs
    • None

    Description

      Current implementation of JsonParserStream uses recursion, which means it can hit StackOverflowError. This is not particularly nice and we have a combined catch with OutOfMemoryError.

      The OOME is something we should not be catching, as when that is raised, we have no idea what happened - it could be us, it could be someone else.

      The SOE is something we can easily avoid by keeping a processing stack.

      Attachments

        # Subject Branch Project Status CR V

        Activity

          People

            Unassigned Unassigned
            rovarga Robert Varga
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated: