Skip to content

Commit

Permalink
MLflow 2.11: Relax primary key constraint on table metrics
Browse files Browse the repository at this point in the history
The test suite started raising `DuplicateKey` exceptions constantly in
concurrent logging situations like exercised on behalf of
`test_log_metric_concurrent_logging_succeeds`.
  • Loading branch information
amotl committed Apr 9, 2024
1 parent 3d0cfe8 commit 7facac1
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions mlflow_cratedb/adapter/ddl/cratedb.sql
Original file line number Diff line number Diff line change
Expand Up @@ -59,8 +59,7 @@ CREATE TABLE IF NOT EXISTS "metrics" (
"timestamp" BIGINT NOT NULL,
"step" BIGINT NOT NULL,
"is_nan" BOOLEAN NOT NULL,
"run_uuid" TEXT NOT NULL,
PRIMARY KEY ("key", "timestamp", "step", "run_uuid", "value", "is_nan")
"run_uuid" TEXT NOT NULL
);

CREATE TABLE IF NOT EXISTS "model_versions" (
Expand Down

0 comments on commit 7facac1

Please sign in to comment.