[
https://issues.apache.org/jira/browse/GEODE-4080?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16293132#comment-16293132
]
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_r157288973
##########
File path:
geode-client-protocol/src/main/java/org/apache/geode/internal/protocol/serialization/TypeConverter.java
##########
@@ -24,10 +24,10 @@
* @param <T> the type this codec knows how to convert
*/
@Experimental
-public interface TypeCodec<T> {
- T decode(byte[] incoming);
+public interface TypeConverter<F, T> {
+ T decode(F incoming);
Review comment:
Should this have "throws EncodingException"? (same for encode)
----------------------------------------------------------------
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)
