abstractdog commented on a change in pull request #161:
URL: https://github.com/apache/tez/pull/161#discussion_r749677672
##########
File path: tez-api/src/main/java/org/apache/tez/dag/api/TezConfiguration.java
##########
@@ -1975,6 +1975,18 @@ public static void validateProperty(String property,
Scope usedScope) {
TEZ_PREFIX + "test.minicluster.app.wait.on.shutdown.secs";
public static final long
TEZ_TEST_MINI_CLUSTER_APP_WAIT_ON_SHUTDOWN_SECS_DEFAULT = 30;
+ /**
+ * Long value
+ * Status Cache timeout window in minutes for the DAGClient.
+ */
+ @Private
+ @ConfigurationScope(Scope.CLIENT)
+ @ConfigurationProperty(type="long")
+ public static final String TEZ_CLIENT_DAG_STATUS_CACHE_TIMEOUT_MINUTES =
TEZ_PREFIX
+ + "client.dag.status.cache.timeout-minutes";
+ // Default timeout is 5 minutes.
+ public static final long TEZ_CLIENT_DAG_STATUS_CACHE_TIMEOUT_MINUTES_DEFAULT
= 5;
Review comment:
I think we can lower this to 1 minute, especially because in case of
query status stuck (that the patch is about to address) this default value will
be the minimum time in which the client has the chance to realize the stuck
cached progress
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
