You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Gerrit0 (via GitHub)" <gi...@apache.org> on 2024/05/31 04:27:11 UTC
[PR] [C++] Do not escape solidus in JSON output [avro]
Gerrit0 opened a new pull request, #2929:
URL: https://github.com/apache/avro/pull/2929
## What is the purpose of the change
The JSON standard permits a solidus (`/`) to be escaped in strings, but does not require that it be escaped. Most JSON serializers used by other packages/languages do not escape solidus characters, so for consistency it would be nice if Avro also did not perform this useless escape.
## Verifying this change
This change is already covered by existing json tests, I've added additional test cases to ensure full coverage.
## Documentation
- Does this pull request introduce a new feature? No
- If yes, how is the feature documented? N/A
--
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: dev-unsubscribe@avro.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [C++] Do not escape solidus in JSON output [avro]
Posted by "martin-g (via GitHub)" <gi...@apache.org>.
martin-g commented on PR #2929:
URL: https://github.com/apache/avro/pull/2929#issuecomment-2155632647
I just noticed there is no Jira ticket for this. We'll need one for the changelog
--
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: issues-unsubscribe@avro.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [C++] Do not escape solidus in JSON output [avro]
Posted by "martin-g (via GitHub)" <gi...@apache.org>.
martin-g merged PR #2929:
URL: https://github.com/apache/avro/pull/2929
--
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: dev-unsubscribe@avro.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [C++] Do not escape solidus in JSON output [avro]
Posted by "Gerrit0 (via GitHub)" <gi...@apache.org>.
Gerrit0 commented on PR #2929:
URL: https://github.com/apache/avro/pull/2929#issuecomment-2156012625
I created [AVRO-3994](https://issues.apache.org/jira/browse/AVRO-3994) to track this change
--
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: issues-unsubscribe@avro.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org