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 2009/10/23 22:39:41 UTC
svn commit: r829216 - in /hadoop/pig/trunk: ./
lib-src/bzip2/org/apache/tools/bzip2r/ src/org/apache/pig/
src/org/apache/pig/backend/executionengine/util/
src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/
src/org/apache/pig/backend/hado...
Author: daijy
Date: Fri Oct 23 20:39:39 2009
New Revision: 829216
URL: http://svn.apache.org/viewvc?rev=829216&view=rev
Log:
PIG-1041: javac warnings: cast, fallthrough, serial
Modified:
hadoop/pig/trunk/CHANGES.txt
hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2InputStream.java
hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2OutputStream.java
hadoop/pig/trunk/src/org/apache/pig/Main.java
hadoop/pig/trunk/src/org/apache/pig/backend/executionengine/util/ExecTools.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/Launcher.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/DotMRPrinter.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/MROperPlan.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogToPhyTranslationVisitor.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogicalToPhysicalTranslatorException.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POBinCond.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POIsNull.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POMapLookUp.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PONegative.java
hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java
hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalExecutionEngine.java
hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalPigLauncher.java
hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/LocalLogToPhyTranslationVisitor.java
hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/relationalOperators/POCogroup.java
hadoop/pig/trunk/src/org/apache/pig/builtin/FloatMax.java
hadoop/pig/trunk/src/org/apache/pig/builtin/FloatSum.java
hadoop/pig/trunk/src/org/apache/pig/builtin/IntMax.java
hadoop/pig/trunk/src/org/apache/pig/builtin/IntSum.java
hadoop/pig/trunk/src/org/apache/pig/builtin/LongSum.java
hadoop/pig/trunk/src/org/apache/pig/builtin/Utf8StorageConverter.java
hadoop/pig/trunk/src/org/apache/pig/data/DataByteArray.java
hadoop/pig/trunk/src/org/apache/pig/data/DefaultAbstractBag.java
hadoop/pig/trunk/src/org/apache/pig/data/SingleTupleBag.java
hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/parser/QueryParser.jjt
hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/validators/TypeCheckerException.java
hadoop/pig/trunk/src/org/apache/pig/impl/plan/PlanException.java
hadoop/pig/trunk/src/org/apache/pig/impl/plan/VisitorException.java
hadoop/pig/trunk/src/org/apache/pig/impl/plan/optimizer/OptimizerException.java
hadoop/pig/trunk/src/org/apache/pig/impl/util/Pair.java
hadoop/pig/trunk/src/org/apache/pig/impl/util/SpillableMemoryManager.java
hadoop/pig/trunk/src/org/apache/pig/pen/DerivedDataVisitor.java
hadoop/pig/trunk/src/org/apache/pig/pen/LineageTrimmingVisitor.java
Modified: hadoop/pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/CHANGES.txt?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/CHANGES.txt (original)
+++ hadoop/pig/trunk/CHANGES.txt Fri Oct 23 20:39:39 2009
@@ -76,6 +76,8 @@
PIG-1033: javac warnings: deprecated hadoop APIs (daijy)
+PIG-1041: javac warnings: cast, fallthrough, serial (daijy)
+
OPTIMIZATIONS
BUG FIXES
Modified: hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2InputStream.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2InputStream.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2InputStream.java (original)
+++ hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2InputStream.java Fri Oct 23 20:39:39 2009
@@ -433,11 +433,11 @@
}
private int bsGetIntVS(int numBits) throws IOException {
- return (int) bsR(numBits);
+ return bsR(numBits);
}
private int bsGetInt32() throws IOException {
- return (int) bsGetint();
+ return bsGetint();
}
private void hbCreateDecodeTables(int[] limit, int[] base,
@@ -796,7 +796,7 @@
}
for (i = 0; i <= last; i++) {
- ch = (char) ll8[i];
+ ch = ll8[i];
tt[cftab[ch]] = i;
cftab[ch]++;
}
@@ -830,7 +830,7 @@
}
}
rNToGo--;
- ch2 ^= (int) ((rNToGo == 1) ? 1 : 0);
+ ch2 ^= ((rNToGo == 1) ? 1 : 0);
i2++;
currentChar = ch2;
Modified: hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2OutputStream.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2OutputStream.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2OutputStream.java (original)
+++ hadoop/pig/trunk/lib-src/bzip2/org/apache/tools/bzip2r/CBZip2OutputStream.java Fri Oct 23 20:39:39 2009
@@ -1211,7 +1211,7 @@
quadrant[i] = 0;
}
- block[0] = (char) (block[last + 1]);
+ block[0] = block[last + 1];
if (last < 4000) {
/*
Modified: hadoop/pig/trunk/src/org/apache/pig/Main.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/Main.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/Main.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/Main.java Fri Oct 23 20:39:39 2009
@@ -537,9 +537,9 @@
final Manifest manifest = jar.getManifest();
final Map <String,Attributes> attrs = manifest.getEntries();
Attributes attr = attrs.get("org/apache/pig");
- String version = (String) attr.getValue("Implementation-Version");
- String svnRevision = (String) attr.getValue("Svn-Revision");
- String buildTime = (String) attr.getValue("Build-TimeStamp");
+ String version = attr.getValue("Implementation-Version");
+ String svnRevision = attr.getValue("Svn-Revision");
+ String buildTime = attr.getValue("Build-TimeStamp");
// we use a version string similar to svn
//svn, version 1.4.4 (r25188)
// compiled Sep 23 2007, 22:32:34
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/executionengine/util/ExecTools.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/executionengine/util/ExecTools.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/executionengine/util/ExecTools.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/executionengine/util/ExecTools.java Fri Oct 23 20:39:39 2009
@@ -41,7 +41,7 @@
PhysicalPlan plan,
PigContext pigContext) throws ExecException {
try {
- PhysicalOperator leaf = (PhysicalOperator)plan.getLeaves().get(0);
+ PhysicalOperator leaf = plan.getLeaves().get(0);
FileSpec spec = null;
if(!(leaf instanceof POStore)){
String scope = leaf.getOperatorKey().getScope();
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/Launcher.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/Launcher.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/Launcher.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/Launcher.java Fri Oct 23 20:39:39 2009
@@ -155,7 +155,7 @@
IOException;
protected boolean isComplete(double prog){
- return (int)(Math.ceil(prog)) == (int)1;
+ return (int)(Math.ceil(prog)) == 1;
}
protected void getStats(Job job, JobClient jobClient, boolean errNotDbg, PigContext pigContext) throws Exception {
@@ -494,7 +494,7 @@
}
StackTraceElement[] steArr = new StackTraceElement[stackTraceElements.size()];
- ((Throwable)object).setStackTrace((StackTraceElement[])(stackTraceElements.toArray(steArr)));
+ ((Throwable)object).setStackTrace(stackTraceElements.toArray(steArr));
if(startingLineNum < (stackTraceLines.length - 1)) {
Throwable e = getExceptionFromStrings(stackTraceLines, startingLineNum);
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/DotMRPrinter.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/DotMRPrinter.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/DotMRPrinter.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/DotMRPrinter.java Fri Oct 23 20:39:39 2009
@@ -111,6 +111,8 @@
* combine phases in an MR operator.
*/
public class InnerOperator extends Operator<PlanVisitor> {
+
+ private static final long serialVersionUID = 1L;
String name;
PhysicalPlan plan;
int code;
@@ -138,6 +140,9 @@
* execution phase.
*/
public class InnerPlan extends OperatorPlan<InnerOperator> {
+
+ private static final long serialVersionUID = 1L;
+
public InnerPlan(PhysicalPlan mapPlan, PhysicalPlan combinePlan,
PhysicalPlan reducePlan) {
try {
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/MROperPlan.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/MROperPlan.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/MROperPlan.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/plans/MROperPlan.java Fri Oct 23 20:39:39 2009
@@ -34,6 +34,8 @@
*/
public class MROperPlan extends OperatorPlan<MapReduceOper> {
+ private static final long serialVersionUID = 1L;
+
public MROperPlan() {
// TODO Auto-generated constructor stub
}
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogToPhyTranslationVisitor.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogToPhyTranslationVisitor.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogToPhyTranslationVisitor.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogToPhyTranslationVisitor.java Fri Oct 23 20:39:39 2009
@@ -944,7 +944,7 @@
.spawnChildWalker(lp);
pushWalker(childWalker);
mCurrentWalker.walk(this);
- exprPlans.add((PhysicalPlan) currentPlan);
+ exprPlans.add(currentPlan);
popWalker();
}
@@ -1178,7 +1178,7 @@
mCurrentWalker.walk(this);
popWalker();
- poFilter.setPlan((PhysicalPlan) currentPlan);
+ poFilter.setPlan(currentPlan);
currentPlan = currentPlans.pop();
List<LogicalOperator> op = filter.getPlan().getPredecessors(filter);
@@ -1327,7 +1327,7 @@
.spawnChildWalker(plan);
pushWalker(childWalker);
childWalker.walk(this);
- sortPlans.add((PhysicalPlan) currentPlan);
+ sortPlans.add(currentPlan);
popWalker();
}
currentPlan = currentPlans.pop();
@@ -1472,7 +1472,7 @@
mCurrentWalker.walk(this);
popWalker();
- ((POFilter) physOp).setPlan((PhysicalPlan) currentPlan);
+ ((POFilter) physOp).setPlan(currentPlan);
currentPlan = currentPlans.pop();
currentPlan.add(physOp);
@@ -1719,7 +1719,7 @@
@Override
public void visit(LOMapLookup op) throws VisitorException {
- String scope = ((OperatorKey) op.getOperatorKey()).scope;
+ String scope = (op.getOperatorKey()).scope;
ExpressionOperator physOp = new POMapLookUp(new OperatorKey(scope,
nodeGen.getNextNodeId(scope)), op.getRequestedParallelism(), op
.getLookUpKey());
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogicalToPhysicalTranslatorException.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogicalToPhysicalTranslatorException.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogicalToPhysicalTranslatorException.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/LogicalToPhysicalTranslatorException.java Fri Oct 23 20:39:39 2009
@@ -21,6 +21,8 @@
public class LogicalToPhysicalTranslatorException extends VisitorException {
+ private static final long serialVersionUID = 1L;
+
/**
* Create a new LogicalToPhysicalTranslatorException with null as the error message.
*/
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POBinCond.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POBinCond.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POBinCond.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POBinCond.java Fri Oct 23 20:39:39 2009
@@ -32,6 +32,8 @@
import org.apache.pig.impl.plan.VisitorException;
public class POBinCond extends ExpressionOperator {
+
+ private static final long serialVersionUID = 1L;
ExpressionOperator cond;
ExpressionOperator lhs;
ExpressionOperator rhs;
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POIsNull.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POIsNull.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POIsNull.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POIsNull.java Fri Oct 23 20:39:39 2009
@@ -35,6 +35,8 @@
public class POIsNull extends UnaryComparisonOperator {
+ private static final long serialVersionUID = 1L;
+
public POIsNull(OperatorKey k, int rp) {
super(k, rp);
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POMapLookUp.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POMapLookUp.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POMapLookUp.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POMapLookUp.java Fri Oct 23 20:39:39 2009
@@ -33,7 +33,8 @@
public class POMapLookUp extends ExpressionOperator {
- private String key;
+ private static final long serialVersionUID = 1L;
+ private String key;
public POMapLookUp(OperatorKey k) {
super(k);
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PONegative.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PONegative.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PONegative.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/PONegative.java Fri Oct 23 20:39:39 2009
@@ -28,6 +28,8 @@
public class PONegative extends UnaryExpressionOperator {
+ private static final long serialVersionUID = 1L;
+
public PONegative(OperatorKey k, int rp) {
super(k, rp);
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POForEach.java Fri Oct 23 20:39:39 2009
@@ -409,7 +409,7 @@
planLeafOps = new PhysicalOperator[inputPlans.size()];
for(PhysicalPlan p : inputPlans) {
++i;
- PhysicalOperator leaf = (PhysicalOperator)p.getLeaves().get(0);
+ PhysicalOperator leaf = p.getLeaves().get(0);
planLeafOps[i] = leaf;
if(leaf instanceof POProject &&
leaf.getResultType() == DataType.TUPLE &&
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalExecutionEngine.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalExecutionEngine.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalExecutionEngine.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalExecutionEngine.java Fri Oct 23 20:39:39 2009
@@ -147,7 +147,7 @@
public List<ExecJob> execute(PhysicalPlan plan, String jobName)
throws ExecException {
try {
- PhysicalOperator leaf = (PhysicalOperator) plan.getLeaves().get(0);
+ PhysicalOperator leaf = plan.getLeaves().get(0);
if (!(leaf instanceof POStore)) {
String scope = leaf.getOperatorKey().getScope();
POStore str = new POStore(new OperatorKey(scope,
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalPigLauncher.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalPigLauncher.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalPigLauncher.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/LocalPigLauncher.java Fri Oct 23 20:39:39 2009
@@ -135,6 +135,7 @@
}
+ @SuppressWarnings("fallthrough")
private int runPipeline(POStore[] leaves, PigContext pc) throws IOException, ExecException {
BitSet bs = new BitSet(leaves.length);
int failed = 0;
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/LocalLogToPhyTranslationVisitor.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/LocalLogToPhyTranslationVisitor.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/LocalLogToPhyTranslationVisitor.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/LocalLogToPhyTranslationVisitor.java Fri Oct 23 20:39:39 2009
@@ -96,7 +96,7 @@
.spawnChildWalker(lp);
pushWalker(childWalker);
mCurrentWalker.walk(this);
- exprPlans.add((PhysicalPlan) currentPlan);
+ exprPlans.add(currentPlan);
popWalker();
}
@@ -284,7 +284,7 @@
mCurrentWalker.walk(this);
popWalker();
- ((POSplitOutput) physOp).setPlan((PhysicalPlan) currentPlan);
+ ((POSplitOutput) physOp).setPlan(currentPlan);
currentPlan = currentPlans.pop();
currentPlan.add(physOp);
PhysicalOperator from = logToPhyMap.get(split.getPlan()
Modified: hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/relationalOperators/POCogroup.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/relationalOperators/POCogroup.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/relationalOperators/POCogroup.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/backend/local/executionengine/physicalLayer/relationalOperators/POCogroup.java Fri Oct 23 20:39:39 2009
@@ -206,7 +206,7 @@
t = data[i];
continue; //since the previous data was probably null so we dont really need a comparison
}
- if(comp.compare(t, (Tuple) data[i]) > 0)
+ if(comp.compare(t, data[i]) > 0)
t = data[i];
}
return t;
Modified: hadoop/pig/trunk/src/org/apache/pig/builtin/FloatMax.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/builtin/FloatMax.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/builtin/FloatMax.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/builtin/FloatMax.java Fri Oct 23 20:39:39 2009
@@ -128,7 +128,7 @@
float curMax = Float.NEGATIVE_INFINITY;
boolean sawNonNull = false;
for (Iterator<Tuple> it = values.iterator(); it.hasNext();) {
- Tuple t = (Tuple)it.next();
+ Tuple t = it.next();
try {
Float f = (Float)(t.get(0));
if (f == null) continue;
Modified: hadoop/pig/trunk/src/org/apache/pig/builtin/FloatSum.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/builtin/FloatSum.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/builtin/FloatSum.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/builtin/FloatSum.java Fri Oct 23 20:39:39 2009
@@ -136,7 +136,7 @@
double sum = 0;
boolean sawNonNull = false;
for (Iterator<Tuple> it = values.iterator(); it.hasNext();) {
- Tuple t = (Tuple) it.next();
+ Tuple t = it.next();
try {
Double d = (Double)(t.get(0));
if (d == null) continue;
@@ -170,7 +170,7 @@
double sum = 0.0;
boolean sawNonNull = false;
for (Iterator<Tuple> it = values.iterator(); it.hasNext();) {
- Tuple t = (Tuple) it.next();
+ Tuple t = it.next();
try {
Float f = (Float)(t.get(0));
if (f == null) continue;
Modified: hadoop/pig/trunk/src/org/apache/pig/builtin/IntMax.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/builtin/IntMax.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/builtin/IntMax.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/builtin/IntMax.java Fri Oct 23 20:39:39 2009
@@ -128,7 +128,7 @@
int curMax = Integer.MIN_VALUE;
boolean sawNonNull = false;
for (Iterator<Tuple> it = values.iterator(); it.hasNext();) {
- Tuple t = (Tuple)it.next();
+ Tuple t = it.next();
try {
Integer i = (Integer)(t.get(0));
if (i == null) continue;
Modified: hadoop/pig/trunk/src/org/apache/pig/builtin/IntSum.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/builtin/IntSum.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/builtin/IntSum.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/builtin/IntSum.java Fri Oct 23 20:39:39 2009
@@ -136,7 +136,7 @@
long sum = 0;
boolean sawNonNull = false;
for (Iterator<Tuple> it = values.iterator(); it.hasNext();) {
- Tuple t = (Tuple) it.next();
+ Tuple t = it.next();
try {
Long l = (Long)(t.get(0));
if (l == null) continue;
@@ -169,7 +169,7 @@
long sum = 0;
boolean sawNonNull = false;
for (Iterator<Tuple> it = values.iterator(); it.hasNext();) {
- Tuple t = (Tuple) it.next();
+ Tuple t = it.next();
try {
Integer i = (Integer)(t.get(0));
if (i == null) continue;
Modified: hadoop/pig/trunk/src/org/apache/pig/builtin/LongSum.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/builtin/LongSum.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/builtin/LongSum.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/builtin/LongSum.java Fri Oct 23 20:39:39 2009
@@ -131,7 +131,7 @@
long sum = 0;
boolean sawNonNull = false;
for (Iterator<Tuple> it = values.iterator(); it.hasNext();) {
- Tuple t = (Tuple) it.next();
+ Tuple t = it.next();
try {
Long l = (Long)(t.get(0));
if (l == null) continue;
Modified: hadoop/pig/trunk/src/org/apache/pig/builtin/Utf8StorageConverter.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/builtin/Utf8StorageConverter.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/builtin/Utf8StorageConverter.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/builtin/Utf8StorageConverter.java Fri Oct 23 20:39:39 2009
@@ -89,7 +89,7 @@
PigWarning.FIELD_DISCARDED_TYPE_CONVERSION_FAILED, mLog);
return null;
}
- return (DataBag)db;
+ return db;
}
public String bytesToCharArray(byte[] b) throws IOException {
Modified: hadoop/pig/trunk/src/org/apache/pig/data/DataByteArray.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/data/DataByteArray.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/data/DataByteArray.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/data/DataByteArray.java Fri Oct 23 20:39:39 2009
@@ -30,6 +30,8 @@
* other methods.
*/
public class DataByteArray implements Comparable, Serializable {
+
+ private static final long serialVersionUID = 1L;
byte[] mData = null;
/**
Modified: hadoop/pig/trunk/src/org/apache/pig/data/DefaultAbstractBag.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/data/DefaultAbstractBag.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/data/DefaultAbstractBag.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/data/DefaultAbstractBag.java Fri Oct 23 20:39:39 2009
@@ -355,9 +355,11 @@
}
public static abstract class BagDelimiterTuple extends DefaultTuple{}
- public static class StartBag extends BagDelimiterTuple{}
+ public static class StartBag extends BagDelimiterTuple{
+ private static final long serialVersionUID = 1L;}
- public static class EndBag extends BagDelimiterTuple{}
+ public static class EndBag extends BagDelimiterTuple{
+ private static final long serialVersionUID = 1L;}
public static final Tuple startBag = new StartBag();
public static final Tuple endBag = new EndBag();
Modified: hadoop/pig/trunk/src/org/apache/pig/data/SingleTupleBag.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/data/SingleTupleBag.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/data/SingleTupleBag.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/data/SingleTupleBag.java Fri Oct 23 20:39:39 2009
@@ -36,7 +36,8 @@
* a single Tuple non-serializable DataBag is required.
*/
public class SingleTupleBag implements DataBag {
-
+
+ private static final long serialVersionUID = 1L;
Tuple item;
public SingleTupleBag(Tuple t) {
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/parser/QueryParser.jjt
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/parser/QueryParser.jjt?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/parser/QueryParser.jjt (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/parser/QueryParser.jjt Fri Oct 23 20:39:39 2009
@@ -1810,7 +1810,7 @@
{
LogicalPlan sortColPlan = new LogicalPlan();
LOProject projectStar = new LOProject(sortColPlan, new OperatorKey(scope, getNextId()), op, -1);
- ((LOProject)projectStar).setStar(true);
+ projectStar.setStar(true);
sortColPlan.add(projectStar);
sortColPlans.add(sortColPlan);
log.debug("Set star to true");
@@ -2605,7 +2605,7 @@
{
LogicalPlan sortColPlan = new LogicalPlan();
LOProject projectStar = new LOProject(sortColPlan, new OperatorKey(scope, getNextId()), eOp, -1);
- ((LOProject)projectStar).setStar(true);
+ projectStar.setStar(true);
sortColPlan.add(projectStar);
sortColPlans.add(sortColPlan);
if(asc) {
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/validators/TypeCheckerException.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/validators/TypeCheckerException.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/validators/TypeCheckerException.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/logicalLayer/validators/TypeCheckerException.java Fri Oct 23 20:39:39 2009
@@ -21,6 +21,8 @@
public class TypeCheckerException extends VisitorException {
+ private static final long serialVersionUID = 1L;
+
/**
* Create a new TypeCheckerException with null as the error message.
*/
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/plan/PlanException.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/plan/PlanException.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/plan/PlanException.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/plan/PlanException.java Fri Oct 23 20:39:39 2009
@@ -21,6 +21,8 @@
public class PlanException extends FrontendException {
+ private static final long serialVersionUID = 1L;
+
/**
* Create a new PlanException with null as the error message.
*/
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/plan/VisitorException.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/plan/VisitorException.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/plan/VisitorException.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/plan/VisitorException.java Fri Oct 23 20:39:39 2009
@@ -21,6 +21,8 @@
public class VisitorException extends FrontendException {
+ private static final long serialVersionUID = 1L;
+
/**
* Create a new VisitorException with null as the error message.
*/
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/plan/optimizer/OptimizerException.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/plan/optimizer/OptimizerException.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/plan/optimizer/OptimizerException.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/plan/optimizer/OptimizerException.java Fri Oct 23 20:39:39 2009
@@ -22,6 +22,8 @@
public class OptimizerException extends VisitorException {
+ private static final long serialVersionUID = 1L;
+
/**
* Create a new OptimizerException with null as the error message.
*/
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/util/Pair.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/util/Pair.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/util/Pair.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/util/Pair.java Fri Oct 23 20:39:39 2009
@@ -23,6 +23,8 @@
* Copy of C++ STL pair container.
*/
public class Pair<T, U> implements Serializable {
+
+ private static final long serialVersionUID = 1L;
public T first;
public U second;
Modified: hadoop/pig/trunk/src/org/apache/pig/impl/util/SpillableMemoryManager.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/impl/util/SpillableMemoryManager.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/impl/util/SpillableMemoryManager.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/impl/util/SpillableMemoryManager.java Fri Oct 23 20:39:39 2009
@@ -119,10 +119,10 @@
try {
spillFileSizeThreshold = Long.parseLong(
- (String) properties.getProperty("pig.spill.size.threshold") ) ;
+ properties.getProperty("pig.spill.size.threshold") ) ;
gcActivationSize = Long.parseLong(
- (String) properties.getProperty("pig.spill.gc.activation.size") ) ;
+ properties.getProperty("pig.spill.gc.activation.size") ) ;
}
catch (NumberFormatException nfe) {
throw new RuntimeException("Error while converting system configurations" +
Modified: hadoop/pig/trunk/src/org/apache/pig/pen/DerivedDataVisitor.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/pen/DerivedDataVisitor.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/pen/DerivedDataVisitor.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/pen/DerivedDataVisitor.java Fri Oct 23 20:39:39 2009
@@ -433,7 +433,7 @@
// replace the original inputs by POReads
for (int i = 0; i < inputs.size(); i++) {
- DataBag bag = derivedData.get(((LOCogroup) op).getInputs().get(i));
+ DataBag bag = derivedData.get(op.getInputs().get(i));
PORead por = new PORead(new OperatorKey("", r.nextLong()), bag);
phy.add(por);
try {
Modified: hadoop/pig/trunk/src/org/apache/pig/pen/LineageTrimmingVisitor.java
URL: http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/pen/LineageTrimmingVisitor.java?rev=829216&r1=829215&r2=829216&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/pen/LineageTrimmingVisitor.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/pen/LineageTrimmingVisitor.java Fri Oct 23 20:39:39 2009
@@ -338,8 +338,7 @@
Set<IdentityHashSet<Tuple>> equivClassesCovered = lineageGroupToEquivClasses
.get(lineageGroup);
int numEquivClassesCovered = equivClassesCovered.size();
- double score = ((double) numEquivClassesCovered)
- / ((double) weight);
+ double score = ((double) numEquivClassesCovered) / weight;
if (score > bestScore) {