发布

  • [OPIK-5624] [BE/FE] fix: restore assertion_results in experiment items after evaluation suite deletion (#6064)

    frostbyte_neo 发布于 2026-04-02 17:17:01 +00:00

    • [OPIK-5624] [FE] fix: show Results column when evaluation suite is deleted but assertion_results exist

    • [OPIK-5624] [BE] fix: restore assertion_results in experiment items after evaluation suite deletion

    When an evaluation suite dataset is deleted from MySQL, the ClickHouse
    dataset_item_versions records remain. Experiments store their dataset_version_id
    at creation time, so the versioned DAO query can still find the correct items and
    return assertion_results correctly.

    • DatasetItemVersionDAO: change versionId parameter from UUID to String so an empty
      string can be used as a safe fallback when no MySQL version exists; add
      handleSqlError to gracefully recover from ClickHouse JSONPath BAD_ARGUMENTS errors
    • DatasetItemService: use empty-string fallback instead of switching to the legacy
      non-versioned DAO when no versions are found; experiments with an explicit
      dataset_version_id stored in ClickHouse will still resolve correctly via COALESCE
    • DatasetService: remove stale workspace-fallback side-effect from resolveDatasetByName
    • DatasetsResource: wrap the pre-stream resolveDatasetByName call in a try/catch so a
      NotFoundException from visibility checks does not escape to Jersey and cause HTTP 404
      on the streaming endpoint; return an empty stream instead (HTTP 200, no entity)
    • InternalErrorMessageBodyWriter: add MessageBodyWriter for ErrorMessage with
      application/octet-stream so streaming-endpoint error responses serialise correctly
    • DatasetsResourceTest: add regression test for assertion_results after suite deletion
      and fix expectedItem in the existing deletion test to match ClickHouse-retained data
    • revert(fe): remove unnecessary frontend changes for OPIK-5624

    • fix: restore workspace fallback header on streaming endpoint and revert unnecessary FE changes

    下载附件