Set task wait timeout to 5 mins by default and docs fixes (facebookin…

…cubator#7911) Summary: Pull Request resolved: facebookincubator#7911 Reviewed By: mbasmanova Differential Revision: D51927483 Pulled By: xiaoxmeng fbshipit-source-id: 78ba34a28b57eb0971bed835263f460eec589069
mbasmanova · Dec 7, 2023 · 13d54c4 · 13d54c4
1 parent debe03c
commit 13d54c4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/velox/common/memory/Memory.h b/velox/common/memory/Memory.h
@@ -116,8 +116,8 @@ struct MemoryManagerOptions {
 
   /// Specifies the max time to wait for memory reclaim by arbitration. The
   /// memory reclaim might fail if the max wait time has exceeded. If it is
-  /// zero, then there is no timeout.
-  uint64_t memoryReclaimWaitMs{0};
+  /// zero, then there is no timeout. The default is 5 mins.
+  uint64_t memoryReclaimWaitMs{300'000};
 
   /// Provided by the query system to validate the state after a memory pool
   /// enters arbitration if not null. For instance, Prestissimo provides

diff --git a/velox/docs/index.rst b/velox/docs/index.rst
@@ -10,7 +10,7 @@ Velox Documentation
     functions
     spark_functions
     configs
-    stats
+    metrics
     bindings/python/README_generated_pyvelox
     develop
     programming-guide
diff --git a/velox/docs/stats.rst → velox/docs/metrics.rst b/velox/docs/stats.rst → velox/docs/metrics.rst
@@ -26,6 +26,8 @@ implementation of BaseStatsReporter is required to integrate with a specific
 monitoring service. The metric aggregation granularity and export interval are
 also configured based on the actual used monitoring service.
 
+Velox supports five metric types:
+
 **Count**: tracks the count of events, such as the number of query failures.
 
 **Sum**: tracks the sum of event data point values, such as sum of query scan