hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Xuefu Zhang" <xzh...@cloudera.com>
Subject Review Request 14590: HIVE-5520: Use factory methods to instantiate HiveDecimal instead of constructors
Date Fri, 11 Oct 2013 05:28:57 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14590/
-----------------------------------------------------------

Review request for hive and Ashutosh Chauhan.


Bugs: HIVE-5520
    https://issues.apache.org/jira/browse/HIVE-5520


Repository: hive-git


Description
-------

Refactoring the code to use static factory methods to replace existing constructors.


Diffs
-----

  common/src/java/org/apache/hadoop/hive/common/type/HiveDecimal.java e5a35cd 
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/ColumnStatisticsImpl.java 42d897c 
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/RecordReaderImpl.java fe1845c 
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFAverage.java 6caef3f 
  ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestOrcFile.java 1dfcb56 
  ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestOrcSerDeStats.java bd9ea21 
  ql/src/test/org/apache/hadoop/hive/ql/udf/TestGenericUDFAbs.java 3b4a7ac 
  serde/src/java/org/apache/hadoop/hive/serde2/RegexSerDe.java add5bdf 
  serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDe.java 5b18c8f

  serde/src/java/org/apache/hadoop/hive/serde2/io/HiveDecimalWritable.java 81500a8 
  serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyHiveDecimal.java 08f251c 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/JavaHiveDecimalObjectInspector.java
92e47fa 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorUtils.java
9a93740 
  serde/src/test/org/apache/hadoop/hive/serde2/binarysortable/TestBinarySortableSerDe.java
82feeec 
  serde/src/test/org/apache/hadoop/hive/serde2/io/TestTimestampWritable.java e54c105 
  serde/src/test/org/apache/hadoop/hive/serde2/objectinspector/TestObjectInspectorConverters.java
5e546c5 

Diff: https://reviews.apache.org/r/14590/diff/


Testing
-------

No new tests necessary. Waiting for pre-commit test results.


Thanks,

Xuefu Zhang


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message