Skip to content

Conversation

@forLp811
Copy link
Contributor

@forLp811 forLp811 commented Nov 22, 2022

Why are the changes needed?

Fix dockerfile image missing RELEASE causes connection kyuubi Times: No such file or directory

How was this patch tested?

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

  • Add screenshots for manual tests if appropriate

  • Run test locally before make a pull request

liaopeng and others added 2 commits November 22, 2022 17:56
Fix dockerfile image missing RELEASE causes connection kyuubi Times: No such file or directory
@pan3793 pan3793 changed the title Update Dockerfile Update Dockerfile to include LICENSE NOTICE RELEASE files Nov 22, 2022
@pan3793
Copy link
Member

pan3793 commented Nov 22, 2022

Thanks, merging to master/1.6

@pan3793 pan3793 added this to the v1.6.2 milestone Nov 22, 2022
@pan3793 pan3793 closed this in c989af6 Nov 22, 2022
pan3793 pushed a commit that referenced this pull request Nov 22, 2022
### _Why are the changes needed?_

Fix dockerfile image missing RELEASE causes connection kyuubi Times: No such file or directory

### _How was this patch tested?_
- [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

- [ ] Add screenshots for manual tests if appropriate

- [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #3831 from forLp811/master.

Closes #3831

0396688 [liaopeng] Merge remote-tracking branch 'origin/master'
715b903 [forLp811] Update Dockerfile
76129f5 [liaopeng] Fix dockerfile image missing RELEASE causes connection kyuubi Times: No such file or directory

Lead-authored-by: liaopeng <Lp%$2wsx@@@@>
Co-authored-by: forLp811 <118807263+forLp811@users.noreply.github.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit c989af6)
Signed-off-by: Cheng Pan <chengpan@apache.org>
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.

3 participants