marius/Open_Ultra_SAST
1
0
Fork 0
mirror of https://github.com/norandom/OpenUltraSAST.git synced 2026-10-16 11:49:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(cpg): count requests, not the census key, when reporting a split retry

Browse Source 
The warning once read 22126 of 22125 requests answered in parts, because the census rides the
answer under its own key and was counted as a request. An instrument that miscounts by one casts
doubt on every number beside it.

Co-Authored-By: Claude Fable 5.1 <[email protected]>
Claude-Session: https://claude.ai/code/session_01EmM1j46qgKMh3zqD1QMwEo
This commit is contained in:
Marius Ciepluch
2026-09-22 22:02:38 +02:00
co-authored by Claude Fable 5.1
parent a084dce780
commit e3edf11cf8
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/openultrasast/cpg/backend.py
+5 -1
View File
@@ -1163,7 +1163,11 @@ class JoernBackend:
merged.setdefault(rid, rows)
if recovered:
self._note("batch_split_after_failure:" + query)
logger.warning("cpg batch %s failed whole; %d of %d requests answered in parts", query, len(merged), len(requests))
# Requests only. The census rides the answer under its own key and is not a request; counting it
# once printed "22126 of 22125 requests answered", and an instrument that miscounts by one casts
# doubt on every number beside it.
answered_requests = sum(1 for rid in merged if rid in requests)
logger.warning("cpg batch %s failed whole; %d of %d requests answered in parts", query, answered_requests, len(requests))
return merged if recovered else None
def _batch_once(self, cpg_path: Path, query: str, requests: Mapping[str, Mapping[str, object]]) -> dict[str, list[object]] | None: