You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ha...@apache.org on 2016/07/29 00:21:30 UTC

hive git commit: HIVE-11339 : org.apache.hadoop.hive.serde2.io.TimestampWritable.write(DataOutput out) makes incorrect cast (Zoltan Haindrich via Ashutosh Chauhan)

Repository: hive
Updated Branches:
  refs/heads/master 24b39cdf2 -> b7543e120


HIVE-11339 : org.apache.hadoop.hive.serde2.io.TimestampWritable.write(DataOutput out) makes incorrect cast (Zoltan Haindrich via Ashutosh Chauhan)

Signed-off-by: Ashutosh Chauhan <ha...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/b7543e12
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/b7543e12
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/b7543e12

Branch: refs/heads/master
Commit: b7543e120a532d3c536d1880836ff1e0b2b43cb1
Parents: 24b39cd
Author: Zoltan Haindrich <ki...@rxd.hu>
Authored: Wed Apr 27 07:37:00 2016 -0800
Committer: Ashutosh Chauhan <ha...@apache.org>
Committed: Thu Jul 28 17:20:15 2016 -0700

----------------------------------------------------------------------
 .../apache/hadoop/hive/accumulo/mr/TestHiveAccumuloTypes.java  | 2 +-
 .../org/apache/hadoop/hive/serde2/io/TimestampWritable.java    | 6 +-----
 2 files changed, 2 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/b7543e12/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/mr/TestHiveAccumuloTypes.java
----------------------------------------------------------------------
diff --git a/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/mr/TestHiveAccumuloTypes.java b/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/mr/TestHiveAccumuloTypes.java
index a378535..926f572 100644
--- a/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/mr/TestHiveAccumuloTypes.java
+++ b/accumulo-handler/src/test/org/apache/hadoop/hive/accumulo/mr/TestHiveAccumuloTypes.java
@@ -245,7 +245,7 @@ public class TestHiveAccumuloTypes {
     Timestamp timestampValue = new Timestamp(now.getTime());
     ByteStream.Output output = new ByteStream.Output();
     TimestampWritable timestampWritable = new TimestampWritable(new Timestamp(now.getTime()));
-    timestampWritable.write(output);
+    timestampWritable.write(new DataOutputStream(output));
     output.close();
     m.put(cfBytes, "timestamp".getBytes(), output.toByteArray());
 

http://git-wip-us.apache.org/repos/asf/hive/blob/b7543e12/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java
----------------------------------------------------------------------
diff --git a/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java b/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java
index 7d136b4..b90e576 100644
--- a/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java
+++ b/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java
@@ -357,15 +357,11 @@ public class TimestampWritable implements WritableComparable<TimestampWritable>
     this.offset = 0;
   }
 
-  public void write(OutputStream out) throws IOException {
+  public void write(DataOutput out) throws IOException {
     checkBytes();
     out.write(currentBytes, offset, getTotalLength());
   }
 
-  public void write(DataOutput out) throws IOException {
-    write((OutputStream) out);
-  }
-
   public int compareTo(TimestampWritable t) {
     checkBytes();
     long s1 = this.getSeconds();