Namespace validation

[kk-08]

Purpose / Goal

Adds namespace validation for XML

Closes #265

Type

• Bug Fix
• Refactoring / Technology upgrade
• New Feature

Description

Perf test results:

Before
validation : 12267.12026907211 requests/second
xml to json : 15042.8615171619 requests/second
xml to json + json string : 13716.406269165005 requests/second
xml to json string : 2662.81536578866 requests/second
xml2js : 5438.29834726833 requests/second

After
validation : 13867.26753315293 requests/second
xml to json : 16933.889511481724 requests/second
xml to json + json string : 15117.827735598856 requests/second
xml to json string : 2567.4248020602818 requests/second
xml2js : 4739.0545687362255 requests/second

[coveralls]

Coverage increased (+0.4%) to 94.301% when pulling ca8ffd5 on kk-08:feature/nsValidation into c665954 on NaturalIntelligence:master.

[coveralls]

Coverage increased (+0.3%) to 94.18% when pulling 3d15e7f on kk-08:feature/nsValidation into c665954 on NaturalIntelligence:master.

[amitguptagwl]

Thanks for the PR. I'll try to test your changes ASAP.

[amitguptagwl]

I have given a few initial comments. Please check

[kk-08]

@amitguptagwl Pushed a few changes. These should improve the performance as well. Please check

[kk-08]

@amitguptagwl Suggested changes pushed. Please check and update.

[kk-08]

@amitguptagwl Any updates on this?

[amitguptagwl]

@kk-08 Sorry for the late response. Please check v4 branch that I have pushed recently. As I had already done these changes, I'm avoiding reviewing the code for same purpose.

The changes in v4 are almost done. But I've not completed all the tests yet. As soon I migrate all the tests from master branch to v4 branch. I would be in the condition to publish them. However, there may be some additional changes before going live but that decision can be taken later.

[amitguptagwl]

@kk-08 v4 is live now. And changes seem useful, could you please fix the conflicts so I can review and merge the PR.