CharlesKWON
dd270d5d9d
Systematic pass through PRG hot paths, promoting them to Go RTL while
preserving Harbour/FiveSql2 semantics. Full log in
docs/RTL-Go-Native-Migration.md.
Bench (bench_sql) vs 2026-04-08 baseline
- B1 SELECT * 2,192 → 114 µs (19x)
- B6 INNER JOIN 9,291 → 233 µs (40x)
- B7 CTE simple 8,037 → 129 µs (62x)
- B9 ROW_NUMBER 3,705 → 265 µs (14x)
- B10 RANK PARTITION 4,748 → 309 µs (15x)
- B12 INSERT (WA cache) 4,319 → 63 µs (69x)
- B13 UPDATE (WA cache) 6,144 → 68 µs (90x)
- B15 CTE+WIN+JOIN 18,395 → 1,873 µs (10x)
Infrastructure
- HbHash O(1) Index preserving insertion order (Harbour KEEPORDER)
- HbDeepClone Go RTL (scalar-sharing, immutable hash keys)
- MEMRDD auto-imported via gengo; all Five programs get mem:name driver
- SQL plan + pcode caches (s_hPlanCache, s_hDmlPcodeCache)
- Opt-in SqlWACacheEnable — dbUseArea/Close/Commit batched for DML
SQL engine
- FiveSql2 lexer ported to Go (byte FSM) with combined automatic
template parameterization (literals → ?, concat queries share plan)
- Go RTL: SqlDistinct, SqlGroupRows, SqlWindowPartitions,
SqlWindowSortPartition, SqlWindowAssignRank, SqlComputeAggSimple,
SqlBulkInsert, SqlBulkUpdate, SqlExprHasAgg, SqlEvalHaving
- CTE / subquery / driving-table materialize paths use MEMRDD
- SqlCoerce/SqlCmp/SqlIsTrue helpers moved from PRG to Go
- SqlBulkUpdate defers Flush when WA cache active (APFS fsync was
dominant B13 cost — 1.6ms/call → gone)
Correctness fixes uncovered during migration
- ASort default path now sorts dates/logicals/timestamps (was no-op)
- ORDER BY default NULL placement matches PRG SqlRowCompare across
Go fast path; explicit NULLS FIRST/LAST honored by both paths
- SqlBulkUpdate respects EXCLUSIVE vs SHARED mode record locks
- SqlCmp/SqlCmpEq normalize NumInt vs Double (caught by test 6b)
Verification
- go test ./... ALL PASS
- FiveSql2 test_sql1999 43/43
- tests/compat_harbour 56/56 (+5 new: ASort dates/logicals,
AScan int cross-type)
- Regression test test_null_order.prg for ORDER BY NULL ordering
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
77 lines
2.0 KiB
Plaintext
77 lines
2.0 KiB
Plaintext
// Large-scale UPDATE benchmark — many matching rows so the per-row
|
|
// savings of SqlBulkUpdate amortize the PcCompile setup cost.
|
|
|
|
#include "FiveSqlDef.ch"
|
|
|
|
PROCEDURE Main()
|
|
|
|
LOCAL t0, t1, i
|
|
|
|
ErrorBlock( {|e| QOut( "TRAP: " + e:description + " " + e:operation ), Break(e) } )
|
|
|
|
? "================================================================"
|
|
? " FiveSql2 UPDATE Benchmark (10k rows, many matching)"
|
|
? "================================================================"
|
|
?
|
|
|
|
SetupLarge()
|
|
|
|
/* Match 2500 rows per UPDATE (val BETWEEN 2500 AND 5000). */
|
|
t0 := hb_MilliSeconds()
|
|
FOR i := 1 TO 50
|
|
five_SQL( "UPDATE bench_big SET val = val + 1 WHERE val BETWEEN 2500 AND 5000" )
|
|
NEXT
|
|
t1 := hb_MilliSeconds()
|
|
R( "UPD_2500match_50iter", t1 - t0 )
|
|
|
|
/* Match ALL rows (no WHERE). */
|
|
t0 := hb_MilliSeconds()
|
|
FOR i := 1 TO 10
|
|
five_SQL( "UPDATE bench_big SET val = val + 0" )
|
|
NEXT
|
|
t1 := hb_MilliSeconds()
|
|
R( "UPD_all_10iter_10k_each", t1 - t0 )
|
|
|
|
/* Match 0 rows (WHERE never true). */
|
|
t0 := hb_MilliSeconds()
|
|
FOR i := 1 TO 100
|
|
five_SQL( "UPDATE bench_big SET val = val + 1 WHERE val < 0" )
|
|
NEXT
|
|
t1 := hb_MilliSeconds()
|
|
R( "UPD_0match_100iter", t1 - t0 )
|
|
|
|
CleanupLarge()
|
|
?
|
|
? "================================================================"
|
|
RETURN
|
|
|
|
STATIC PROCEDURE SetupLarge()
|
|
LOCAL i
|
|
IF hb_FileExists( "bench_big.dbf" )
|
|
FErase( "bench_big.dbf" )
|
|
ENDIF
|
|
dbCreate( "bench_big.dbf", { ;
|
|
{ "ID", "N", 10, 0 }, ;
|
|
{ "NAME", "C", 30, 0 }, ;
|
|
{ "VAL", "N", 10, 0 } ;
|
|
} )
|
|
USE bench_big.dbf NEW EXCLUSIVE
|
|
FOR i := 1 TO 10000
|
|
dbAppend()
|
|
FieldPut( 1, i )
|
|
FieldPut( 2, "N_" + PadL( hb_ntos( i ), 6, "0" ) )
|
|
FieldPut( 3, i )
|
|
NEXT
|
|
dbCommit()
|
|
CLOSE bench_big
|
|
RETURN
|
|
|
|
STATIC PROCEDURE CleanupLarge()
|
|
dbCloseAll()
|
|
FErase( "bench_big.dbf" )
|
|
RETURN
|
|
|
|
STATIC FUNCTION R( cLabel, nMs )
|
|
? " ", PadR( cLabel, 32 ) + Str( nMs, 7 ) + " ms"
|
|
RETURN NIL
|