Github user GJL commented on a diff in the pull request:
https://github.com/apache/flink/pull/5093#discussion_r153483875
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/messages/json/ResourceIDSerializer.java
---
@@ -29,16 +30,16 @@
/**
* Json serializer for {@link InstanceID}.
*/
-public class InstanceIDSerializer extends StdSerializer<InstanceID> {
+public class ResourceIDSerializer extends StdSerializer<ResourceID> {
private static final long serialVersionUID = 5798852092159615938L;
- protected InstanceIDSerializer() {
- super(InstanceID.class);
+ protected ResourceIDSerializer() {
+ super(ResourceID.class);
}
@Override
- public void serialize(InstanceID value, JsonGenerator gen,
SerializerProvider provider) throws IOException {
+ public void serialize(ResourceID value, JsonGenerator gen,
SerializerProvider provider) throws IOException {
gen.writeString(value.toString());
--- End diff --
Maybe `value.getResourceIdString()`.
---
