You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@avro.apache.org by sc...@apache.org on 2012/02/24 23:43:32 UTC

svn commit: r1293464 - in /avro/trunk: CHANGES.txt lang/java/avro/src/main/java/org/apache/avro/Schema.java

Author: scottcarey
Date: Fri Feb 24 22:43:32 2012
New Revision: 1293464

URL: http://svn.apache.org/viewvc?rev=1293464&view=rev
Log:
AVRO-1037. Problems using Avro 1.6.2 with Hadoop (CDH3 or 1.0)

Modified:
    avro/trunk/CHANGES.txt
    avro/trunk/lang/java/avro/src/main/java/org/apache/avro/Schema.java

Modified: avro/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/avro/trunk/CHANGES.txt?rev=1293464&r1=1293463&r2=1293464&view=diff
==============================================================================
--- avro/trunk/CHANGES.txt (original)
+++ avro/trunk/CHANGES.txt Fri Feb 24 22:43:32 2012
@@ -10,6 +10,8 @@ Avro 1.6.3 (unreleased)
 
   BUG FIXES
 
+    AVRO-1037. Problems using Avro 1.6.2 with Hadoop (CDH3 or 1.0) (scottcarey)
+
 Avro 1.6.2 (13 February 2012)
 
   NEW FEATURES

Modified: avro/trunk/lang/java/avro/src/main/java/org/apache/avro/Schema.java
URL: http://svn.apache.org/viewvc/avro/trunk/lang/java/avro/src/main/java/org/apache/avro/Schema.java?rev=1293464&r1=1293463&r2=1293464&view=diff
==============================================================================
--- avro/trunk/lang/java/avro/src/main/java/org/apache/avro/Schema.java (original)
+++ avro/trunk/lang/java/avro/src/main/java/org/apache/avro/Schema.java Fri Feb 24 22:43:32 2012
@@ -458,8 +458,8 @@ public abstract class Schema {
     private boolean defaultValueEquals(JsonNode thatDefaultValue) {
       if (defaultValue == null)
         return thatDefaultValue == null;
-      if (Double.isNaN(defaultValue.getValueAsDouble()))
-        return Double.isNaN(thatDefaultValue.getValueAsDouble());
+      if (Double.isNaN(defaultValue.getDoubleValue()))
+        return Double.isNaN(thatDefaultValue.getDoubleValue());
       return defaultValue.equals(thatDefaultValue);
     }