Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: databricks/databricks-sql-python
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: databricks/databricks-sql-python
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: ai/bump-engine-654a31d0
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 2 commits
  • 2 files changed
  • 1 contributor

Commits on Aug 13, 2026

  1. chore(bots): bump engine pin d05dcb11 -> 654a31d0 

    Move all bots to engine main (654a31d0), 30 commits ahead of the current pin.
This repo already runs the modern daily-cron retrospective + closed loop; this
brings the pinned engine in line with the sibling driver repos (kernel + the
go/nodejs/jdbc/odbc learning PRs), whose engine was bumped to the same SHA.

The main addition since d05dcb11 is per-bot model selection (default opus-4.8).
SDK/CLI locked versions (0.2.102 / 2.1.61) are unchanged at 654a31d0, so this is
a SHA-only bump — no install-bot-engine edit needed.

Co-authored-by: Isaac
Signed-off-by: eric-wang-1990 <e.wang@databricks.com>
    @eric-wang-1990
    eric-wang-1990 committed Aug 13, 2026
    Configuration menu
    Copy the full SHA
    77349ef View commit details
    Browse the repository at this point in the history

  2. fix(learning): correct MODEL_ENDPOINT to the concrete serving-endpoin… 

    …t form

The learning workflow set MODEL_ENDPOINT to
`.../serving-endpoints/anthropic/invocations`. That is NOT translated the way the
old comment claimed: sdk_agent.translate_endpoint has an early-return for URLs
already containing `/serving-endpoints/anthropic`, which keeps the trailing
`/invocations` — so the CLI appends `/v1/messages` and calls
`.../serving-endpoints/anthropic/invocations/v1/messages`, which the gateway
rejects with `400 Unsupported native API path`.

Proof: databricks-sql-python's learning cron (same endpoint form) has failed
every scheduled run with exactly this 400. The reviewer-bot flagged it on the
sibling PRs (Low), and it is the real cause.

Use the concrete `.../serving-endpoints/databricks-claude-opus-4-8/invocations`
form that reviewer-bot.yml / engineer-bot.yml already use successfully:
translate_endpoint strips `<model>/invocations` to the `.../serving-endpoints/
anthropic` base the CLI needs. Comment corrected to explain the trap.

Co-authored-by: Isaac
Signed-off-by: eric-wang-1990 <e.wang@databricks.com>
    @eric-wang-1990
    eric-wang-1990 committed Aug 13, 2026
    Configuration menu
    Copy the full SHA
    31ffc17 View commit details
    Browse the repository at this point in the history
Loading