Skip to content

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