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

Upgrade playground to Kyuubi 1.7.1 #5266

Closed
wants to merge 5 commits into from

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Sep 8, 2023

Why are the changes needed?

Upgrade to latest Kyuubi in playground.

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

    Successfully published docker image via BUILDX=1 docker/playground/build-image.sh

  • Add screenshots for manual tests if appropriate

  • Run test locally before make a pull request

Was this patch authored or co-authored using generative AI tooling?

No.

@pan3793 pan3793 self-assigned this Sep 8, 2023
@pan3793 pan3793 added this to the v1.9.0 milestone Sep 8, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #5266 (17015e4) into master (2371cc6) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #5266   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         590     590           
  Lines       33345   33345           
  Branches     4406    4406           
======================================
  Misses      33345   33345           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

📢 Have feedback on the report? Share it here.

@pan3793 pan3793 closed this in 8c5fadf Sep 11, 2023
@pan3793
Copy link
Member Author

pan3793 commented Sep 11, 2023

Thanks, merged to master(1.9)

@pan3793 pan3793 deleted the playground-171 branch October 7, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants