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

Add support for PythonWheelTask #872

Merged
merged 3 commits into from
Oct 29, 2021

Conversation

tingting-gao-db
Copy link
Contributor

WIP - to be tested

Copy link
Contributor

@nfx nfx left a comment

Choose a reason for hiding this comment

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

please reformat code and also add few lines to documentation in https://github.com/databrickslabs/terraform-provider-databricks/blob/master/docs/resources/job.md, so that users would know of the feature.

you can reuse the HCL example you've been testing with in docs.

compute/model.go Outdated
@@ -517,6 +517,14 @@ type SparkSubmitTask struct {
Parameters []string `json:"parameters,omitempty"`
}

// PythonWheelTask contains the information for python wheel jobs
type PythonWheelTask struct {
EntryPoint string `json:"entry_point,omitempty"`
Copy link
Contributor

Choose a reason for hiding this comment

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

please run make fmt and commit :)

@codecov
Copy link

codecov bot commented Oct 26, 2021

Codecov Report

Merging #872 (75b969c) into master (d7f276f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #872   +/-   ##
=======================================
  Coverage   83.80%   83.80%           
=======================================
  Files          92       92           
  Lines        8233     8233           
=======================================
  Hits         6900     6900           
  Misses        842      842           
  Partials      491      491           
Impacted Files Coverage Δ
compute/model.go 83.33% <ø> (ø)

@nfx nfx added this to the v0.3.10 milestone Oct 28, 2021
@nfx nfx merged commit b1976a3 into databricks:master Oct 29, 2021
@nfx nfx mentioned this pull request Oct 29, 2021
michael-berk pushed a commit to michael-berk/terraform-provider-databricks that referenced this pull request Feb 15, 2023
* Add support for PythonWheelTask

Co-authored-by: Serge Smertin <259697+nfx@users.noreply.github.com>
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.

2 participants