Skip to content

Commit e333923

Browse files
authored
feat: enhance resource scheduling logs with clear status and configuration details (#18684)
* improve: enhance queue scheduling logs with clear status and configuration details - Show specific configuration parameters (max_concurrency, max_running_queries) - Display current resource usage (X/Y slots used) - Include queue length and wait times for better visibility - Distinguish between workload group and warehouse limits with clear [BLOCKED]/[ACQUIRED] status * fix: use saturating_sub to prevent underflow in permit calculation - Replace subtraction with saturating_sub to avoid potential underflow - Simplify variable naming for better readability - Ensure thread-safe calculation of used slots * fix: update all status messages to use clear BLOCKED format - Replace remaining generic status messages with specific blocked status - Update workload group global semaphore acquisition message - Ensure all code paths show detailed configuration information * test: add workload group concurrency status logging test - Add test to verify status messages during workload group concurrency limits - Simulate 100 concurrent queries with max_concurrency=16 - Capture and log status messages for debugging queue behavior * clean: remove test case and unused imports * feat: add queries waiting count to all BLOCKED status messages - Add queue length to workload group local limit messages - Add queue length to workload group global limit messages - Ensure consistency with warehouse limit messages that already show waiting count * refactor: change BLOCKED to WAITING for friendlier status messages - Replace [BLOCKED] with [WAITING] prefix for all queue status messages - More user-friendly indication that queries are waiting rather than blocked
1 parent 3f8a918 commit e333923

File tree

6 files changed

+245
-224
lines changed

6 files changed

+245
-224
lines changed

scripts/ci/ci-run-sqllogic-tests-cluster-nginx.sh

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@
44

55
set -e
66

7-
87
echo "*************************************"
98
echo "* Setting STORAGE_TYPE to S3. *"
109
echo "* *"

0 commit comments

Comments
 (0)