Commit History
-
[NOTICKET][DB] fix(query): R2 - always compile a bounded LIMIT
Rifqi Hafizuddin +1: Claude Opus 4.8An IR with no limit compiled to a LIMIT-less SELECT, and DbExecutor materialized the entire result set before capping at 10k rows - an unbounded fetch that can OOM on a large user table. - SqlCompiler now bounds every query: an explicit IR limit is honored (clamped to MAX_RESULT_ROWS=10000); an unbounded query gets LIMIT cap+1 so the executor can distinguish exactly-the-cap from more-rows-existed and flag truncation. - CompiledSql.row_cap carries the effective cap; DbExecutor caps + derives truncated from it and drops its own _ROW_HARD_CAP constant. - test_sql.py: no-limit golden strings get the safety bound; +3 cases (no-limit bounded, explicit-limit clamp, row_cap). Restored S608 to tests/** ruff ignore (documented in PR3-DB but missing from pyproject) - golden tests assert literal SQL. Also logs R13: a pre-existing, unrelated test_prompt failure found while running R2. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>6974c5b
-
feat/Catalog Retrieval System (#1)
- initial commit (8557a2045ca2c0873e07efdf6227064b83e4a1e5) - initial structure (efc0c0a8b2444014cbe74f04617c3c974d9915be) - [KM-553] initialize shared contract (f31f6731692b0199d8cb5c5d29dffa466d6d4ca0) - [KM-553] migrate retrieval layer and remove obsolete rag/tools modules (df220ea47c96eb8a1efa07c6eebef4604fc308d6) - [NOTICKET] catalog enricher + structured pipeline + on_db_registered (77ea82d29d86400c5ed30ac154be8df6a7e897eb) - Merge commit 'refs/pr/1' of https://huggingface.co/spaces/DataEyond/Agentic-Service-Data-Eyond-Catalog into pr/1 (0ae8717c7efce2cda14fe4b8caecd774459b1f6e) - [NOTICKET] SqlCompiler (Postgres) + DbExecutor (212dad360714508c1aa607995039e6818fa40f78) - Merge commit 'refs/pr/1' of https://huggingface.co/spaces/DataEyond/Agentic-Service-Data-Eyond-Catalog into pr/1 (faf8a4f6d7e06ccf1ce84c2e7f4990c79f27f843) - TabularIntrospector + on_tabular_uploaded trigger + 31 tests (ca425208969b72a20ed76f86ef0faa1a0e912339) - [KM 556] add columns field to QueryResult + update…6bff5d9
