You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by da...@apache.org on 2014/10/14 00:01:39 UTC
svn commit: r1631575 - in /pig/branches/branch-0.14: CHANGES.txt
contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/MultiStorage.java
Author: daijy
Date: Mon Oct 13 22:01:39 2014
New Revision: 1631575
URL: http://svn.apache.org/r1631575
Log:
PIG-2834: MultiStorage requires unused constructor argument
Modified:
pig/branches/branch-0.14/CHANGES.txt
pig/branches/branch-0.14/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/MultiStorage.java
Modified: pig/branches/branch-0.14/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.14/CHANGES.txt?rev=1631575&r1=1631574&r2=1631575&view=diff
==============================================================================
--- pig/branches/branch-0.14/CHANGES.txt (original)
+++ pig/branches/branch-0.14/CHANGES.txt Mon Oct 13 22:01:39 2014
@@ -84,6 +84,8 @@ OPTIMIZATIONS
BUG FIXES
+PIG-2834: MultiStorage requires unused constructor argument (daijy)
+
PIG-4230: Documentation fix: first nested foreach example is incomplete (lbendig via daijy)
PIG-3979: group all performance, garbage collection, and incremental aggregation (ddreyfus via daijy)
Modified: pig/branches/branch-0.14/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/MultiStorage.java
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.14/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/MultiStorage.java?rev=1631575&r1=1631574&r2=1631575&view=diff
==============================================================================
--- pig/branches/branch-0.14/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/MultiStorage.java (original)
+++ pig/branches/branch-0.14/contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/MultiStorage.java Mon Oct 13 22:01:39 2014
@@ -99,7 +99,9 @@ public class MultiStorage extends StoreF
* Constructor
*
* @param parentPathStr
- * Parent output dir path
+ * Parent output dir path (this will be specified in store statement,
+ * so MultiStorage don't use this parameter in reality. However, we don't
+ * want to change the construct to break backward compatibility)
* @param splitFieldIndex
* key field index
* @param compression