Skip to content

[2.0] Add searchExperiments API to Java client#6579

Merged
dbczumar merged 1 commit intomlflow:branch-2.0from
dbczumar:searchexp_java_20
Aug 25, 2022
Merged

[2.0] Add searchExperiments API to Java client#6579
dbczumar merged 1 commit intomlflow:branch-2.0from
dbczumar:searchexp_java_20

Conversation

@dbczumar
Copy link
Copy Markdown
Collaborator

Signed-off-by: dbczumar [email protected]

What changes are proposed in this pull request?

Add searchExperiments API to Java client

How is this patch tested?

Added integration tests

Does this PR change the documentation?

  • No. You can skip the rest of this section.
  • Yes. Make sure the changed pages / sections render correctly by following the steps below.
  1. Click the Details link on the Preview docs check.
  2. Find the changed pages / sections and make sure they render correctly.

Release Notes

Add searchExperiments API to Java client

Is this a user-facing change?

  • No. You can skip the rest of this section.
  • Yes. Give a description of this change to be included in the release notes for MLflow users.

What component(s), interfaces, languages, and integrations does this PR affect?

Components

  • area/artifacts: Artifact stores and artifact logging
  • area/build: Build and test infrastructure for MLflow
  • area/docs: MLflow documentation pages
  • area/examples: Example code
  • area/model-registry: Model Registry service, APIs, and the fluent client calls for Model Registry
  • area/models: MLmodel format, model serialization/deserialization, flavors
  • area/pipelines: Pipelines, Pipeline APIs, Pipeline configs, Pipeline Templates
  • area/projects: MLproject format, project running backends
  • area/scoring: MLflow Model server, model deployment tools, Spark UDFs
  • area/server-infra: MLflow Tracking server backend
  • area/tracking: Tracking Service, tracking client APIs, autologging

Interface

  • area/uiux: Front-end, user experience, plotting, JavaScript, JavaScript dev server
  • area/docker: Docker use across MLflow's components, such as MLflow Projects and MLflow Models
  • area/sqlalchemy: Use of SQLAlchemy in the Tracking Service or Model Registry
  • area/windows: Windows support

Language

  • language/r: R APIs and clients
  • language/java: Java APIs and clients
  • language/new: Proposals for new client languages

Integrations

  • integrations/azure: Azure and Azure ML integrations
  • integrations/sagemaker: SageMaker integrations
  • integrations/databricks: Databricks integrations

How should the PR be classified in the release notes? Choose one:

  • rn/breaking-change - The PR will be mentioned in the "Breaking Changes" section
  • rn/none - No description will be included. The PR will be mentioned only by the PR number in the "Small Bugfixes and Documentation Updates" section
  • rn/feature - A new user-facing feature worth mentioning in the release notes
  • rn/bug-fix - A user-facing bug fix worth mentioning in the release notes
  • rn/documentation - A user-facing documentation change worth mentioning in the release notes

Signed-off-by: dbczumar <[email protected]>
@dbczumar dbczumar requested a review from harupy August 24, 2022 19:33
@github-actions github-actions bot added area/tracking Tracking service, tracking client APIs, autologging language/java Java APIs and clients rn/feature Mention under Features in Changelogs. labels Aug 24, 2022
Copy link
Copy Markdown
Member

@harupy harupy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@dbczumar dbczumar merged commit 504b329 into mlflow:branch-2.0 Aug 25, 2022
tomasatdatabricks pushed a commit to tomasatdatabricks/mlflow that referenced this pull request Sep 2, 2022
Signed-off-by: tomasatdatabricks <[email protected]>
dbczumar added a commit that referenced this pull request Sep 27, 2022
* Update scoring api to wrap response in a dictionary with predictions field and nest dataframe inputs explicit fields

Signed-off-by: tomasatdatabricks <[email protected]>

* fix tests.

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* fix azureml test

Signed-off-by: tomasatdatabricks <[email protected]>

* fix azureml test

Signed-off-by: tomasatdatabricks <[email protected]>

* fix tests

Signed-off-by: tomasatdatabricks <[email protected]>

* fix tests

Signed-off-by: tomasatdatabricks <[email protected]>

* fix rfunc

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* Update mlflow/utils/proto_json_utils.py

Co-authored-by: Corey Zumar <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Update mlflow/utils/proto_json_utils.py

Co-authored-by: Corey Zumar <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Update mlflow/utils/proto_json_utils.py

Co-authored-by: Corey Zumar <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Update mlflow/utils/proto_json_utils.py

Co-authored-by: Corey Zumar <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Fixes.

Signed-off-by: tomasatdatabricks <[email protected]>

* Fix Java scoring.

Signed-off-by: tomasatdatabricks <[email protected]>

* Fixes.

Signed-off-by: tomasatdatabricks <[email protected]>

* [2.0] Add mlflow_search_experiments to R client (#6578)

Signed-off-by: tomasatdatabricks <[email protected]>

* Java files (#6579)

Signed-off-by: tomasatdatabricks <[email protected]>

* [branch-2.0] Set pyfunc.spark_udf env_manager argument default value to be local (#6640)

Signed-off-by: Weichen Xu <[email protected]>

Signed-off-by: Weichen Xu <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Fix env_manager doc (#6655)

Signed-off-by: harupy <[email protected]>

Signed-off-by: harupy <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* fix base ref (#6656)

Signed-off-by: harupy <[email protected]>

Signed-off-by: harupy <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* remove use_conda (#6654)

Signed-off-by: harupy <[email protected]>

Signed-off-by: harupy <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Merged with upstream.

Signed-off-by: tomasatdatabricks <[email protected]>

* Use virtualenv if no env file is present in project (#6658)

* Use virtualenv if no env file is present in project

Signed-off-by: harupy <[email protected]>

* docstring

Signed-off-by: harupy <[email protected]>

* fix tests

Signed-off-by: harupy <[email protected]>

Signed-off-by: harupy <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Remove conda from R client (#6638)

* remove conda environment

Signed-off-by: harupy <[email protected]>

* add Prerequisites

Signed-off-by: harupy <[email protected]>

* remove conda

Signed-off-by: harupy <[email protected]>

* remove test for conda

Signed-off-by: harupy <[email protected]>

* add single quote

Signed-off-by: harupy <[email protected]>

* define environment variables in test step

Signed-off-by: harupy <[email protected]>

* link

Signed-off-by: harupy <[email protected]>

* update document.R

Signed-off-by: harupy <[email protected]>

* do not create conda env

Signed-off-by: harupy <[email protected]>

* rename

Signed-off-by: harupy <[email protected]>

* fix filename

Signed-off-by: harupy <[email protected]>

* remove install.R

Signed-off-by: harupy <[email protected]>

* fix mlflow/R/mlflow/NAMESPACE

Signed-off-by: harupy <[email protected]>

* update doc

Signed-off-by: harupy <[email protected]>

* update README

Signed-off-by: harupy <[email protected]>

* update doc again

Signed-off-by: harupy <[email protected]>

* reset examples/pipelines/sklearn_regression

Signed-off-by: harupy <[email protected]>

* remove reticulate

Signed-off-by: harupy <[email protected]>

* add reticulate in Suggests

Signed-off-by: harupy <[email protected]>

* quote

Signed-off-by: harupy <[email protected]>

* Remove reticulate::use_condaenv

Signed-off-by: harupy <[email protected]>

* find python and mlflow binaries automatically

Signed-off-by: harupy <[email protected]>

* remove MLFLOW_PYTHON_BIN and MLFLOW_BIN

Signed-off-by: harupy <[email protected]>

* fix doc

Signed-off-by: harupy <[email protected]>

* udpate doc

Signed-off-by: harupy <[email protected]>

* fix syntax error

Signed-off-by: harupy <[email protected]>

* fix rst_header

Signed-off-by: harupy <[email protected]>

* revert MLFLOW_CONDA_HOME

Signed-off-by: harupy <[email protected]>

* replace NULL with NA

Signed-off-by: harupy <[email protected]>

* Fix README

Signed-off-by: harupy <[email protected]>

* update R-api.rst

Signed-off-by: harupy <[email protected]>

* use git rev-parse

Signed-off-by: harupy <[email protected]>

* Address comments

Signed-off-by: harupy <[email protected]>

Signed-off-by: harupy <[email protected]>
Signed-off-by: tomasatdatabricks <[email protected]>

* Fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* Fix

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* Add debug print

Signed-off-by: tomasatdatabricks <[email protected]>

* Fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* Java fixes.

Signed-off-by: tomasatdatabricks <[email protected]>

* Python test fixes.

Signed-off-by: tomasatdatabricks <[email protected]>

* Fixed spark udf tests.

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* fix java

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* fix

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* fixes

Signed-off-by: tomasatdatabricks <[email protected]>

* Change output get predictions API, docs

Signed-off-by: dbczumar <[email protected]>

* Lint

Signed-off-by: dbczumar <[email protected]>

* Move imports, docs

Signed-off-by: dbczumar <[email protected]>

* Test coverage

Signed-off-by: dbczumar <[email protected]>

* Format

Signed-off-by: dbczumar <[email protected]>

* Rename

Signed-off-by: dbczumar <[email protected]>

* Rename

Signed-off-by: dbczumar <[email protected]>

* format

Signed-off-by: dbczumar <[email protected]>

* Tuck imports

Signed-off-by: dbczumar <[email protected]>

* Fix naming

Signed-off-by: dbczumar <[email protected]>

* fix docs

Signed-off-by: dbczumar <[email protected]>

* Fix some tests, naming

Signed-off-by: dbczumar <[email protected]>

* Import fixes

Signed-off-by: dbczumar <[email protected]>

* format

Signed-off-by: dbczumar <[email protected]>

* Fix

Signed-off-by: dbczumar <[email protected]>

* Test case fix

Signed-off-by: dbczumar <[email protected]>

* Fix

Signed-off-by: dbczumar <[email protected]>

* format

Signed-off-by: dbczumar <[email protected]>

* Fix

Signed-off-by: dbczumar <[email protected]>

* remove import

Signed-off-by: dbczumar <[email protected]>

* Fix

Signed-off-by: dbczumar <[email protected]>

* Test case fix

Signed-off-by: dbczumar <[email protected]>

* Fix error

Signed-off-by: dbczumar <[email protected]>

* Fix

Signed-off-by: dbczumar <[email protected]>

* Test fix

Signed-off-by: dbczumar <[email protected]>

* Fix TF tests

Signed-off-by: dbczumar <[email protected]>

* Keep deployments in skinny

Signed-off-by: dbczumar <[email protected]>

* PredResp partial

Signed-off-by: dbczumar <[email protected]>

* Fix

Signed-off-by: dbczumar <[email protected]>

* Fix imports

Signed-off-by: dbczumar <[email protected]>

* Fix docs

Signed-off-by: dbczumar <[email protected]>

* Fix examples

Signed-off-by: dbczumar <[email protected]>

* Fix err msg

Signed-off-by: dbczumar <[email protected]>

* Fix bad imports

Signed-off-by: dbczumar <[email protected]>

* Autoformat: https://github.com/mlflow/mlflow/actions/runs/3131614304

Signed-off-by: mlflow-automation <[email protected]>

* debug

Signed-off-by: dbczumar <[email protected]>

* Fix

Signed-off-by: dbczumar <[email protected]>

* more tolerant

Signed-off-by: dbczumar <[email protected]>

Signed-off-by: tomasatdatabricks <[email protected]>
Signed-off-by: Weichen Xu <[email protected]>
Signed-off-by: harupy <[email protected]>
Signed-off-by: dbczumar <[email protected]>
Signed-off-by: mlflow-automation <[email protected]>
Co-authored-by: Corey Zumar <[email protected]>
Co-authored-by: WeichenXu <[email protected]>
Co-authored-by: Harutaka Kawamura <[email protected]>
Co-authored-by: dbczumar <[email protected]>
Co-authored-by: mlflow-automation <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tracking Tracking service, tracking client APIs, autologging language/java Java APIs and clients rn/feature Mention under Features in Changelogs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants