[
https://issues.apache.org/jira/browse/GEODE-4080?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16295754#comment-16295754
]
ASF GitHub Bot commented on GEODE-4080:
---------------------------------------
WireBaron commented on a change in pull request #1171: GEODE-4080: Protobuf
JSON objects are in a proto string
URL: https://github.com/apache/geode/pull/1171#discussion_r157615958
##########
File path:
geode-client-protocol/src/main/java/org/apache/geode/internal/protocol/serialization/codec/JsonPdxConverter.java
##########
@@ -17,19 +17,29 @@
import org.apache.geode.annotations.Experimental;
Review comment:
Hmm, curiously I can't seem to add a comment to line 15, but did we want to
rethink the package name now that we're no longer calling these codecs?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Improve JSON encoding for new protocol
> --------------------------------------
>
> Key: GEODE-4080
> URL: https://issues.apache.org/jira/browse/GEODE-4080
> Project: Geode
> Issue Type: Improvement
> Components: client/server
> Reporter: Galen O'Sullivan
>
> One of the encoding types in the new protobuf protocol is called
> {{CustomEncodedValue}}. However, users aren't really free to encode their own
> types. Also, packing JSON into a byte array is going to be harder for users
> than just sending ti as a string. Let's make the JSON type a string and put
> the custom value type in later when we actually have support for custom
> encoding.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
