You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iotdb.apache.org by ro...@apache.org on 2021/08/24 03:42:25 UTC
[iotdb] 03/04: rename timeout in Session
This is an automated email from the ASF dual-hosted git repository.
rong pushed a commit to branch session-pool-optimization-0.12
in repository https://gitbox.apache.org/repos/asf/iotdb.git
commit ead6631ab5d731e976aac070264e6f7d5d70f198
Author: Steve Yurong Su <ro...@apache.org>
AuthorDate: Tue Aug 24 11:26:04 2021 +0800
rename timeout in Session
---
.../src/main/java/org/apache/iotdb/SessionExample.java | 2 +-
.../main/java/org/apache/iotdb/session/Session.java | 18 +++++++++---------
.../test/java/org/apache/iotdb/session/SessionUT.java | 6 +++---
3 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/example/session/src/main/java/org/apache/iotdb/SessionExample.java b/example/session/src/main/java/org/apache/iotdb/SessionExample.java
index a73c90c..623f813 100644
--- a/example/session/src/main/java/org/apache/iotdb/SessionExample.java
+++ b/example/session/src/main/java/org/apache/iotdb/SessionExample.java
@@ -659,7 +659,7 @@ public class SessionExample {
private static void setTimeout() throws StatementExecutionException {
Session tempSession = new Session(LOCAL_HOST, 6667, "root", "root", 10000, 20000);
- tempSession.setTimeout(60000);
+ tempSession.setQueryTimeout(60000);
}
private static void createClusterSession() throws IoTDBConnectionException {
diff --git a/session/src/main/java/org/apache/iotdb/session/Session.java b/session/src/main/java/org/apache/iotdb/session/Session.java
index 66726e9..20079c9 100644
--- a/session/src/main/java/org/apache/iotdb/session/Session.java
+++ b/session/src/main/java/org/apache/iotdb/session/Session.java
@@ -81,7 +81,7 @@ public class Session {
* Timeout of query can be set by users. If not set, default value 0 will be used, which will use
* server configuration.
*/
- private long timeout = 0;
+ private long queryTimeoutInMs = 0;
protected boolean enableRPCCompression;
protected int connectionTimeoutInMs;
@@ -167,7 +167,7 @@ public class Session {
Config.DEFAULT_INITIAL_BUFFER_CAPACITY,
Config.DEFAULT_MAX_FRAME_SIZE,
Config.DEFAULT_CACHE_LEADER_MODE);
- this.timeout = timeoutInMs;
+ this.queryTimeoutInMs = timeoutInMs;
}
public Session(String host, int rpcPort, String username, String password, ZoneId zoneId) {
@@ -576,18 +576,18 @@ public class Session {
public boolean checkTimeseriesExists(String path)
throws IoTDBConnectionException, StatementExecutionException {
- return defaultSessionConnection.checkTimeseriesExists(path, timeout);
+ return defaultSessionConnection.checkTimeseriesExists(path, queryTimeoutInMs);
}
- public void setTimeout(long timeoutInMs) throws StatementExecutionException {
+ public void setQueryTimeout(long timeoutInMs) throws StatementExecutionException {
if (timeoutInMs < 0) {
throw new StatementExecutionException("Timeout must be >= 0, please check and try again.");
}
- this.timeout = timeoutInMs;
+ this.queryTimeoutInMs = timeoutInMs;
}
- public long getTimeout() {
- return timeout;
+ public long getQueryTimeout() {
+ return queryTimeoutInMs;
}
/**
@@ -598,7 +598,7 @@ public class Session {
*/
public SessionDataSet executeQueryStatement(String sql)
throws StatementExecutionException, IoTDBConnectionException {
- return executeStatementMayRedirect(sql, timeout);
+ return executeStatementMayRedirect(sql, queryTimeoutInMs);
}
/**
@@ -640,7 +640,7 @@ public class Session {
e.getEndPoint());
// retry
try {
- return defaultSessionConnection.executeQueryStatement(sql, timeout);
+ return defaultSessionConnection.executeQueryStatement(sql, queryTimeoutInMs);
} catch (RedirectException redirectException) {
logger.error("{} redirect twice", sql, redirectException);
throw new StatementExecutionException(sql + " redirect twice, please try again.");
diff --git a/session/src/test/java/org/apache/iotdb/session/SessionUT.java b/session/src/test/java/org/apache/iotdb/session/SessionUT.java
index bd0ed59..86a2c5b 100644
--- a/session/src/test/java/org/apache/iotdb/session/SessionUT.java
+++ b/session/src/test/java/org/apache/iotdb/session/SessionUT.java
@@ -195,8 +195,8 @@ public class SessionUT {
@Test
public void setTimeout() throws StatementExecutionException {
session = new Session("127.0.0.1", 6667, "root", "root", 10000, 20000);
- Assert.assertEquals(20000, session.getTimeout());
- session.setTimeout(60000);
- Assert.assertEquals(60000, session.getTimeout());
+ Assert.assertEquals(20000, session.getQueryTimeout());
+ session.setQueryTimeout(60000);
+ Assert.assertEquals(60000, session.getQueryTimeout());
}
}