You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by se...@apache.org on 2017/05/11 16:19:17 UTC
[03/13] flink git commit: [hotfix] [core] Minor code cleanups in
JavaSerializer and SerializerTestBase
[hotfix] [core] Minor code cleanups in JavaSerializer and SerializerTestBase
Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/70c48aaa
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/70c48aaa
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/70c48aaa
Branch: refs/heads/master
Commit: 70c48aaa65a33d1a375ffa5838cf0e9532a4c202
Parents: 6f8022e
Author: Stephan Ewen <se...@apache.org>
Authored: Wed May 10 11:28:55 2017 +0200
Committer: Stephan Ewen <se...@apache.org>
Committed: Thu May 11 12:45:51 2017 +0200
----------------------------------------------------------------------
.../java/typeutils/runtime/kryo/JavaSerializer.java | 4 ++--
.../api/common/typeutils/SerializerTestBase.java | 16 ++++++++++++++--
2 files changed, 16 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/flink/blob/70c48aaa/flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/kryo/JavaSerializer.java
----------------------------------------------------------------------
diff --git a/flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/kryo/JavaSerializer.java b/flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/kryo/JavaSerializer.java
index a51647c..711c814 100644
--- a/flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/kryo/JavaSerializer.java
+++ b/flink-core/src/main/java/org/apache/flink/api/java/typeutils/runtime/kryo/JavaSerializer.java
@@ -45,7 +45,7 @@ public class JavaSerializer<T> extends Serializer<T> {
public JavaSerializer() {}
- @SuppressWarnings("unchecked")
+ @SuppressWarnings({"unchecked", "rawtypes"})
@Override
public void write(Kryo kryo, Output output, T o) {
try {
@@ -62,7 +62,7 @@ public class JavaSerializer<T> extends Serializer<T> {
}
}
- @SuppressWarnings("unchecked")
+ @SuppressWarnings({"unchecked", "rawtypes"})
@Override
public T read(Kryo kryo, Input input, Class aClass) {
try {
http://git-wip-us.apache.org/repos/asf/flink/blob/70c48aaa/flink-core/src/test/java/org/apache/flink/api/common/typeutils/SerializerTestBase.java
----------------------------------------------------------------------
diff --git a/flink-core/src/test/java/org/apache/flink/api/common/typeutils/SerializerTestBase.java b/flink-core/src/test/java/org/apache/flink/api/common/typeutils/SerializerTestBase.java
index a846703..f2879ac 100644
--- a/flink-core/src/test/java/org/apache/flink/api/common/typeutils/SerializerTestBase.java
+++ b/flink-core/src/test/java/org/apache/flink/api/common/typeutils/SerializerTestBase.java
@@ -55,7 +55,13 @@ import org.junit.Test;
public abstract class SerializerTestBase<T> extends TestLogger {
protected abstract TypeSerializer<T> createSerializer();
-
+
+ /**
+ * Gets the expected length for the serializer's {@link TypeSerializer#getLength()} method.
+ *
+ * <p>The expected length should be positive, for fix-length data types, or {@code -1} for
+ * variable-length types.
+ */
protected abstract int getLength();
protected abstract Class<T> getTypeClass();
@@ -124,9 +130,15 @@ public abstract class SerializerTestBase<T> extends TestLogger {
@Test
public void testGetLength() {
+ final int len = getLength();
+
+ if (len == 0) {
+ fail("Broken serializer test base - zero length cannot be the expected length");
+ }
+
try {
TypeSerializer<T> serializer = getSerializer();
- assertEquals(getLength(), serializer.getLength());
+ assertEquals(len, serializer.getLength());
}
catch (Exception e) {
System.err.println(e.getMessage());