Skip to content
This repository has been archived by the owner on Oct 2, 2024. It is now read-only.

Ignore test_log_batch_null_metrics #48

Merged
merged 5 commits into from
Sep 25, 2024
Merged

Ignore test_log_batch_null_metrics #48

merged 5 commits into from
Sep 25, 2024

Conversation

nojaf
Copy link
Collaborator

@nojaf nojaf commented Sep 25, 2024

We should skip test_log_batch_null_metrics as there is no way to invoke the store method with the invalid proto counterpart.

Also ignore test_sqlalchemy_store_behaves_as_expected_with_inmemory_sqlite_db

@nojaf nojaf requested a review from dsuhinin September 25, 2024 11:32
Signed-off-by: nojaf <florian.verdonck@outlook.com>
dsuhinin
dsuhinin previously approved these changes Sep 25, 2024
Signed-off-by: nojaf <florian.verdonck@outlook.com>
@nojaf nojaf merged commit 94f6b77 into jgiannuzzi:main Sep 25, 2024
19 checks passed
jgiannuzzi pushed a commit to mlflow/mlflow-go-backend that referenced this pull request Oct 1, 2024
jgiannuzzi/mlflow-go#48

Signed-off-by: nojaf <florian.verdonck@outlook.com>
jgiannuzzi pushed a commit to mlflow/mlflow-go-backend that referenced this pull request Oct 1, 2024
jgiannuzzi/mlflow-go#48

Signed-off-by: nojaf <florian.verdonck@outlook.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants