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

Adding azure-cosmos-package to databricks spark config #972

Merged
merged 3 commits into from
Jan 19, 2023

Conversation

jainr
Copy link
Collaborator

@jainr jainr commented Jan 17, 2023

Adding azue.spark.cosmos package to spark config so it is available to databricks test cluster.

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

@xiaoyongzhu xiaoyongzhu added the safe to test Tag to execute build pipeline for a PR from forked repo label Jan 17, 2023
@xiaoyongzhu
Copy link
Member

I think this PR should be ready as long as the cosmos error is gone. Other failures can be safely ignored.

Copy link
Collaborator

@Yuqing-cat Yuqing-cat left a comment

Choose a reason for hiding this comment

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

The change looks good to me. Let's wait and see if the cosmos failure in CI result is gone.

@jainr jainr merged commit 3859fca into feathr-ai:main Jan 19, 2023
@xiaoyongzhu xiaoyongzhu mentioned this pull request Jan 19, 2023
2 tasks
xiaoyongzhu added a commit to xiaoyongzhu/feathr that referenced this pull request Jan 19, 2023
blrchen added a commit that referenced this pull request Jan 20, 2023
xiaoyongzhu pushed a commit that referenced this pull request Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test Tag to execute build pipeline for a PR from forked repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants