You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by na...@apache.org on 2010/01/29 20:19:23 UTC
svn commit: r904599 - in /hadoop/hive/trunk: CHANGES.txt
ql/src/java/org/apache/hadoop/hive/ql/Driver.java
ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java
ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java
Author: namit
Date: Fri Jan 29 19:19:21 2010
New Revision: 904599
URL: http://svn.apache.org/viewvc?rev=904599&view=rev
Log:
HIVE-1092. Add job counter for conditional tasks
(He Yongqiang via namit)
Modified:
hadoop/hive/trunk/CHANGES.txt
hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java
hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java
hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java
Modified: hadoop/hive/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hive/trunk/CHANGES.txt?rev=904599&r1=904598&r2=904599&view=diff
==============================================================================
--- hadoop/hive/trunk/CHANGES.txt (original)
+++ hadoop/hive/trunk/CHANGES.txt Fri Jan 29 19:19:21 2010
@@ -101,6 +101,9 @@
HIVE-1110. Add a counter to denote that skewjoin got triggered
(He Yongqiang via namit)
+ HIVE-1092. Add job counter for conditional tasks
+ (He Yongqiang via namit)
+
Release 0.5.0 - Unreleased
INCOMPATIBLE CHANGES
@@ -1550,4 +1553,4 @@
HIVE-1087. Let user script write out binary data into a table
(Zheng Shao via Ning Zhang)
-
+
Modified: hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java
URL: http://svn.apache.org/viewvc/hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java?rev=904599&r1=904598&r2=904599&view=diff
==============================================================================
--- hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java (original)
+++ hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/Driver.java Fri Jan 29 19:19:21 2010
@@ -527,8 +527,7 @@
// Launch upto maxthreads tasks
while (runnable.peek() != null && running.size() < maxthreads) {
Task<? extends Serializable> tsk = runnable.remove();
- curJobNo = launchTask(tsk, queryId, noName, running, jobname, jobs,
- curJobNo, driverCxt);
+ launchTask(tsk, queryId, noName, running, jobname, jobs, driverCxt);
}
// poll the Tasks to see which one completed
@@ -628,9 +627,9 @@
* @return the updated number of last the map-reduce job launched
*/
- public int launchTask(Task<? extends Serializable> tsk, String queryId,
+ public void launchTask(Task<? extends Serializable> tsk, String queryId,
boolean noName, Map<TaskResult, TaskRunner> running, String jobname,
- int jobs, int curJobNo, DriverContext cxt) {
+ int jobs, DriverContext cxt) {
if (SessionState.get() != null) {
SessionState.get().getHiveHistory().startTask(queryId, tsk,
@@ -641,8 +640,8 @@
conf.setVar(HiveConf.ConfVars.HADOOPJOBNAME, jobname + "("
+ tsk.getId() + ")");
}
- curJobNo++;
- console.printInfo("Launching Job " + curJobNo + " out of " + jobs);
+ cxt.incCurJobNo(1);
+ console.printInfo("Launching Job " + cxt.getCurJobNo() + " out of " + jobs);
}
tsk.initialize(conf, plan, cxt);
TaskResult tskRes = new TaskResult();
@@ -657,7 +656,7 @@
tskRun.runSequential();
}
running.put(tskRes, tskRun);
- return curJobNo;
+ return;
}
/**
Modified: hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java
URL: http://svn.apache.org/viewvc/hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java?rev=904599&r1=904598&r2=904599&view=diff
==============================================================================
--- hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java (original)
+++ hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/DriverContext.java Fri Jan 29 19:19:21 2010
@@ -27,6 +27,9 @@
public class DriverContext {
Queue<Task<? extends Serializable>> runnable = new LinkedList<Task<? extends Serializable>>();
+
+ // how many jobs have been started
+ int curJobNo;
public DriverContext(Queue<Task<? extends Serializable>> runnable) {
this.runnable = runnable;
@@ -55,4 +58,12 @@
tsk.setQueued();
}
+ public int getCurJobNo() {
+ return curJobNo;
+ }
+
+ public void incCurJobNo(int amount) {
+ this.curJobNo = this.curJobNo + amount;
+ }
+
}
Modified: hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java
URL: http://svn.apache.org/viewvc/hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java?rev=904599&r1=904598&r2=904599&view=diff
==============================================================================
--- hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java (original)
+++ hadoop/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/ConditionalTask.java Fri Jan 29 19:19:21 2010
@@ -86,6 +86,7 @@
console.printInfo(ExecDriver.getJobEndMsg(""
+ Utilities.randGen.nextInt())
+ ", job is filtered out (removed at runtime).");
+ driverContext.incCurJobNo(1);
if (tsk.getChildTasks() != null) {
for (Task<? extends Serializable> child : tsk.getChildTasks()) {
child.parentTasks.remove(tsk);