ctubbsii commented on code in PR #5906:
URL: https://github.com/apache/accumulo/pull/5906#discussion_r2370567191
##########
core/src/main/java/org/apache/accumulo/core/clientImpl/TabletMergeabilityUtil.java:
##########
@@ -80,23 +80,19 @@ public static String encode(Text split, TabletMergeability
tabletMergeability) {
return gson.toJson(jData);
}
- public static ByteBuffer encodeAsBuffer(Pair<Text,TabletMergeability> split)
{
- return ByteBuffer.wrap(encode(split).getBytes(UTF_8));
+ public static ByteBuffer encodeAsBuffer(SplitMergeability sm) {
+ return ByteBuffer.wrap(encode(sm.split(),
sm.mergeability()).getBytes(UTF_8));
}
- public static String encode(Pair<Text,TabletMergeability> split) {
- return encode(split.getFirst(), split.getSecond());
- }
-
- public static Pair<Text,TabletMergeability> decode(ByteBuffer data) {
+ public static SplitMergeability decode(ByteBuffer data) {
return decode(ByteBufferUtil.toString(data));
}
- public static Pair<Text,TabletMergeability> decode(String data) {
+ public static SplitMergeability decode(String data) {
GSonData jData = gson.fromJson(data, GSonData.class);
Review Comment:
I was thinking that they might be able to be consolidated, but didn't look
into it carefully.
--
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]
