You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by pr...@apache.org on 2009/04/17 22:37:41 UTC

svn commit: r766149 - in /hadoop/pig/branches/multiquery/src/org/apache/pig: PigServer.java backend/hadoop/executionengine/physicalLayer/relationalOperators/POSplit.java impl/plan/DotPlanDumper.java impl/plan/PlanDumper.java

Author: pradeepkth
Date: Fri Apr 17 20:37:40 2009
New Revision: 766149

URL: http://svn.apache.org/viewvc?rev=766149&view=rev
Log:
 PIG-627: multiquery support incremental patch to fix javadoc warnings (hagleitn via pradeepkth)

Modified:
    hadoop/pig/branches/multiquery/src/org/apache/pig/PigServer.java
    hadoop/pig/branches/multiquery/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSplit.java
    hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/DotPlanDumper.java
    hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/PlanDumper.java

Modified: hadoop/pig/branches/multiquery/src/org/apache/pig/PigServer.java
URL: http://svn.apache.org/viewvc/hadoop/pig/branches/multiquery/src/org/apache/pig/PigServer.java?rev=766149&r1=766148&r2=766149&view=diff
==============================================================================
--- hadoop/pig/branches/multiquery/src/org/apache/pig/PigServer.java (original)
+++ hadoop/pig/branches/multiquery/src/org/apache/pig/PigServer.java Fri Apr 17 20:37:40 2009
@@ -540,8 +540,8 @@
      * call to execute in the respoect that all the pending stores are
      * marked as complete.
      * @param lps Stream to print the logical tree
-     * @param lps Stream to print the physical tree
-     * @param lps Stream to print the execution tree
+     * @param pps Stream to print the physical tree
+     * @param eps Stream to print the execution tree
      * @throws IOException if the requested alias cannot be found.
      */
     public void explain(String alias,

Modified: hadoop/pig/branches/multiquery/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSplit.java
URL: http://svn.apache.org/viewvc/hadoop/pig/branches/multiquery/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSplit.java?rev=766149&r1=766148&r2=766149&view=diff
==============================================================================
--- hadoop/pig/branches/multiquery/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSplit.java (original)
+++ hadoop/pig/branches/multiquery/src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/relationalOperators/POSplit.java Fri Apr 17 20:37:40 2009
@@ -183,7 +183,7 @@
     /**
      * Appends the specified plan to the end of 
      * the nested input plan list
-     * @param plan plan to be appended to the list
+     * @param inPlan plan to be appended to the list
      */
     public void addPlan(PhysicalPlan inPlan) throws PlanException {        
         myPlans.add(inPlan);

Modified: hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/DotPlanDumper.java
URL: http://svn.apache.org/viewvc/hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/DotPlanDumper.java?rev=766149&r1=766148&r2=766149&view=diff
==============================================================================
--- hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/DotPlanDumper.java (original)
+++ hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/DotPlanDumper.java Fri Apr 17 20:37:40 2009
@@ -164,7 +164,7 @@
 
     /**
      * Used to generate the label for an operator.
-     * @param E operator to dump
+     * @param op operator to dump
      */
     protected String getName(E op) {
         return op.name();
@@ -172,7 +172,7 @@
     
     /**
      * Used to generate the the attributes of a node
-     * @param E operator
+     * @param op operator
      */
     protected String[] getAttributes(E op) {
         String[] attributes = new String[1];

Modified: hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/PlanDumper.java
URL: http://svn.apache.org/viewvc/hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/PlanDumper.java?rev=766149&r1=766148&r2=766149&view=diff
==============================================================================
--- hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/PlanDumper.java (original)
+++ hadoop/pig/branches/multiquery/src/org/apache/pig/impl/plan/PlanDumper.java Fri Apr 17 20:37:40 2009
@@ -84,7 +84,7 @@
     /**
      * makeDumper is a factory method. Used by subclasses to specify
      * what dumper should handle the nested plan.
-     * @param S Plan that the new dumper should handle
+     * @param plan Plan that the new dumper should handle
      * @return the dumper for S
      */
     protected PlanDumper makeDumper(S plan, PrintStream ps) {
@@ -93,7 +93,7 @@
 
     /**
      * Will be called to dump a simple operator
-     * @param E the operator to be dumped
+     * @param op the operator to be dumped
      */
     protected void dumpOperator(E op) {
         ps.println(op.name().replace(" ","_"));
@@ -102,7 +102,7 @@
     /**
      * Will be called when an operator has nested plans, which are
      * connected to one of the multiple inputs.
-     * @param E the nested operator
+     * @param op the nested operator
      * @param plans a map of input operator to connected nested plan
      */
     protected void dumpMultiInputNestedOperator(E op, MultiMap<E,S> plans) {
@@ -118,7 +118,7 @@
     /**
      * Will be called for nested operators. The operators are not
      * specifically connected to any input or output operators of E
-     * @param E the nested operator
+     * @param op the nested operator
      * @param plans a collection of sub plans.
      */
     protected void dumpNestedOperator(E op, Collection<S> plans) {
@@ -142,8 +142,8 @@
     /**
      * Used to determine if an operator has nested plans, which are
      * connected to specific input operators.
-     * @param E operator
-     * @returns Map describing the input to nested plan relationship.
+     * @param op operator
+     * @return Map describing the input to nested plan relationship.
      */
     protected MultiMap<E, S> getMultiInputNestedPlans(E op) {
         return new MultiMap<E, S>();
@@ -152,8 +152,8 @@
     /**
      * Used to determine if an operator has nested plans (without
      * connections to in- or output operators.
-     * @param E operator
-     * @returns Collection of nested plans.
+     * @param op operator
+     * @return Collection of nested plans.
      */
     protected Collection<S> getNestedPlans(E op) {
         return new LinkedList<S>();