report: add queue info for udp

theanarkh · Fyko · commit 21e5d9671370 · 2022-09-15T16:56:12.000-06:00
PR-URL: nodejs#44345 Reviewed-By: Richard Lau <rlau@redhat.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
diff --git a/src/node_report_utils.cc b/src/node_report_utils.cc
@@ -261,7 +261,14 @@ void WalkHandle(uv_handle_t* h, void* arg) {
     writer->json_keyvalue("writable",
                           static_cast<bool>(uv_is_writable(&handle->stream)));
   }
-
+  if (h->type == UV_UDP) {
+    writer->json_keyvalue(
+        "writeQueueSize",
+        uv_udp_get_send_queue_size(reinterpret_cast<uv_udp_t*>(h)));
+    writer->json_keyvalue(
+        "writeQueueCount",
+        uv_udp_get_send_queue_count(reinterpret_cast<uv_udp_t*>(h)));
+  }
   writer->json_end();
 }
 
diff --git a/test/report/test-report-uv-handles.js b/test/report/test-report-uv-handles.js
@@ -260,6 +260,8 @@ if (process.argv[2] === 'child') {
           found_udp.push('connected');
         }
         assert(handle.is_referenced);
+        assert.strictEqual(handle.writeQueueSize, 0);
+        assert.strictEqual(handle.writeQueueCount, 0);
       }, 2),
     };
 

Original file line number	Diff line number	Diff line change
`@@ -260,6 +260,8 @@ if (process.argv[2] === 'child') {`
`260`	`260`	`found_udp.push('connected');`
`261`	`261`	`}`
`262`	`262`	`assert(handle.is_referenced);`
	`263`	`+ assert.strictEqual(handle.writeQueueSize, 0);`
	`264`	`+ assert.strictEqual(handle.writeQueueCount, 0);`
`263`	`265`	`}, 2),`
`264`	`266`	`};`
`265`	`267`