Skip to content

Conversation

massimo-ua
Copy link
Contributor

Purpose / Goal

This is a documentation update regarding to the new XML to JSON parser option alwaysCreateTextNode

Type

Please mention the type of PR

  • [ ]Bug Fix
  • [ ]Refactoring / Technology upgrade
  • [x]New Feature

Issue

349

Perfomance tests

Before

Running Suite: XML Parser benchmark
validation : 22482.344697710287 requests/second
xml to json : 15002.244861535037 requests/second
xml to json + json string : 13868.767986614608 requests/second
xml to json string : 2939.2827347641673 requests/second
xml2js : 5709.279811841135 requests/second

After

Running Suite: XML Parser benchmark
validation : 22732.947506604152 requests/second
xml to json : 15270.010037240734 requests/second
xml to json + json string : 14049.966577736843 requests/second
xml to json string : 2906.0090047436674 requests/second
xml2js : 5690.710124230496 requests/second

@coveralls
Copy link

coveralls commented Oct 15, 2021

Coverage Status

Coverage remained the same at 87.9% when pulling d658d4f on massimo-ua:develop into 352dd89 on NaturalIntelligence:master.

@massimo-ua massimo-ua changed the title Develop Added alwaysCreateTextNode option description Oct 15, 2021
@amitguptagwl amitguptagwl merged commit 003c69e into NaturalIntelligence:master Oct 15, 2021
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