From 0f8f61a4d8926e9a7f7d818369324771b4347f3d Mon Sep 17 00:00:00 2001 From: Joachim Rosskopf Date: Mon, 31 Aug 2026 11:53:20 +0200 Subject: [PATCH] chore: bump erpl-bics and erpl-odp for the tuning API and test hardening erpl-odp (#8): - Guards a null local state that crashed sap_odp_read_full at threads := 8, when DuckDB opens more workers than there are packages. - Gives erpl_odp its first settings: erpl_odp_fetch_size and erpl_odp_max_threads, matching erpl_rfc's vocabulary. I_MAXPACKAGESIZE was compiled in at 2 MiB and unreachable, so nobody could trade round-trips against per-packet memory. - Replaces 17 assertions of the form `count(*) >= 0 ---- true` -- true of any result, including an empty one. Three delta tests passed end-to-end if read_delta always returned nothing. erpl-bics (#11): - sap_bics_begin's rows / columns / filters were declared and never read; they now place characteristics at open time, which also removes three full session round-trips from building a query. - Models the fixed cost of a fetch, which was assumed to be zero, and exposes both cost terms as settings. Budget defaults deliberately unchanged. - The overflow message now states what a refused query will really cost: memory scales with result ROWS, not data cells, so the cell-derived figure it used to quote understated a 46,755-row result by 19x. Suites green at these commits: bics 45/45, odp 24/24 on both backends. --- bics | 2 +- odp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/bics b/bics index 85bb720..e1aea74 160000 --- a/bics +++ b/bics @@ -1 +1 @@ -Subproject commit 85bb7203ebc4aca1d5f68d04ea0c7937a6546edb +Subproject commit e1aea74837cbaca8c42d1be56f25ea7bb4768655 diff --git a/odp b/odp index f5fe9a3..5374ad6 160000 --- a/odp +++ b/odp @@ -1 +1 @@ -Subproject commit f5fe9a39bd2bdda3b7fd84d56230a67f45186fee +Subproject commit 5374ad6c985ad85a4cae5787e3a8d9f5cf09a444