You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by ap...@apache.org on 2019/01/08 20:13:57 UTC
[incubator-pinot] branch master updated: [TE] task - yet another
backlog gauge fix (#3666)
This is an automated email from the ASF dual-hosted git repository.
apucher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new 842678d [TE] task - yet another backlog gauge fix (#3666)
842678d is described below
commit 842678dfcabb8ddc1b1d76df3edf595597ea7006
Author: Alexander Pucher <ap...@linkedin.com>
AuthorDate: Tue Jan 8 12:13:52 2019 -0800
[TE] task - yet another backlog gauge fix (#3666)
(override due to travis blocker)
---
.../java/com/linkedin/thirdeye/datalayer/bao/jdbc/TaskManagerImpl.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/thirdeye/thirdeye-pinot/src/main/java/com/linkedin/thirdeye/datalayer/bao/jdbc/TaskManagerImpl.java b/thirdeye/thirdeye-pinot/src/main/java/com/linkedin/thirdeye/datalayer/bao/jdbc/TaskManagerImpl.java
index 42782aa..e886c08 100644
--- a/thirdeye/thirdeye-pinot/src/main/java/com/linkedin/thirdeye/datalayer/bao/jdbc/TaskManagerImpl.java
+++ b/thirdeye/thirdeye-pinot/src/main/java/com/linkedin/thirdeye/datalayer/bao/jdbc/TaskManagerImpl.java
@@ -177,7 +177,8 @@ public class TaskManagerImpl extends AbstractManagerImpl<TaskDTO> implements Tas
try (Connection connection = this.genericPojoDao.getConnection();
PreparedStatement statement = connection.prepareStatement(COUNT_WAITING_TASKS);
ResultSet rs = statement.executeQuery()){
- return rs.getInt(0);
+ rs.next();
+ return rs.getInt(1);
} catch (Exception e) {
LOG.warn("Could not retrieve task backlog size. Defaulting to -1.", e);
return -1;
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org