Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make use of the new name filtering in calls to list jobs. #595

Merged

Conversation

dinowernli
Copy link
Contributor

@dinowernli dinowernli commented Dec 27, 2022

Tried this out manually:

> databricks jobs list --name=test_continuous --version=2.1
371294009417906  test_continuous

And with 2.0 instead of 2.1:

> databricks jobs list --name=test_continuous
WARN: Your CLI is configured to use Jobs API 2.0. In order to use the latest Jobs features please upgrade to 2.1: 'databricks jobs configure --version=2.1'. Future versions of this CLI will default to the new Jobs API. Learn more at https://docs.databricks.com/dev-tools/cli/jobs-cli.html
ERROR: the options --expand-tasks, --offset, --limit, --all, and --name are only available in API 2.1

@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch from 0900233 to 11deb07 Compare December 29, 2022 14:29
@codecov-commenter
Copy link

codecov-commenter commented Dec 29, 2022

Codecov Report

Base: 75.80% // Head: 75.81% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (202ceca) compared to base (fac68b7).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #595   +/-   ##
=======================================
  Coverage   75.80%   75.81%           
=======================================
  Files          55       55           
  Lines        4922     4924    +2     
=======================================
+ Hits         3731     3733    +2     
  Misses       1191     1191           
Impacted Files Coverage Δ
databricks_cli/jobs/api.py 95.65% <100.00%> (ø)
databricks_cli/jobs/cli.py 96.12% <100.00%> (+0.05%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch 2 times, most recently from 4d5671c to af5fec7 Compare December 29, 2022 14:35
@dinowernli dinowernli marked this pull request as ready for review December 29, 2022 14:36
Copy link
Contributor

@lennartkats-db lennartkats-db left a comment

Choose a reason for hiding this comment

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

Please note that you need to enable commit signing for this repository.

@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch from af5fec7 to b673f82 Compare January 3, 2023 12:50
@dinowernli dinowernli force-pushed the add-name-filter-to-jobs-list branch from b673f82 to 202ceca Compare January 3, 2023 12:56
@lennartkats-db lennartkats-db merged commit fd2ad70 into databricks:main Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants