dzamo commented on pull request #2283:
URL: https://github.com/apache/drill/pull/2283#issuecomment-891687681


   Now that I test master again after this merge, both the self-closing and 
long form empty XML element cases work perfectly.  As with the JSON example 
above, an empty `<foo></foo>` followed by a parent `<foo><bar>1</bar></foo>` 
results in a empty map for the `<foo></foo>`, while before the merge I got an 
error.  I have to confess that I couldn't see all of this getting sorted about 
by this PR, which seemed focussed on self-closing tags only, but perhaps I was 
testing an old build in the first place.  Nice one!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to