Skip to content
This repository was archived by the owner on Jan 28, 2022. It is now read-only.

Conversation

amrasem
Copy link

@amrasem amrasem commented Dec 18, 2019

The bug is about updating current djob with new value for instance updating the notebook path, It doesn't update the notebook path
To reproduce the issue:
Create a djob
Update the notebook path
It doesn't update the notebook path but shows the successful message.

sync between master and forked branch_1712
@msftclas
Copy link

msftclas commented Dec 18, 2019

CLA assistant check
All CLA requirements met.

Copy link
Contributor

@Azadehkhojandi Azadehkhojandi left a comment

Choose a reason for hiding this comment

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

Thanks for submitting PR, great idea of using Databricks Reset on updates.
I put few comments on the files and also could you please add test to cover your update manifest fix?

Extended databricks operator to report metrics into Prometheus (Azure#104)
@Azadehkhojandi
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@Azadehkhojandi Azadehkhojandi left a comment

Choose a reason for hiding this comment

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

Thanks for your PR, I have updated my comments.

@amrasem amrasem force-pushed the fix-90-updating-manifest-djob branch from 27f707a to 09fd8ec Compare February 17, 2020 03:19
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.

6 participants