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

Revert "[KYUUBI #4305][Bug] Backport HIVE-15820: comment at the head … #4406

Closed
wants to merge 1 commit into from

Conversation

cfmcgrady
Copy link
Contributor

…of beeline -e"

This reverts commit c489e29.

Why are the changes needed?

#4333 (comment)

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

@cfmcgrady
Copy link
Contributor Author

cc @pan3793 @cxzl25 @lsm1

@codecov-commenter
Copy link

Codecov Report

Merging #4406 (c7fff72) into master (86dc59c) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #4406      +/-   ##
============================================
- Coverage     53.81%   53.76%   -0.05%     
  Complexity       13       13              
============================================
  Files           564      564              
  Lines         30929    30927       -2     
  Branches       4163     4163              
============================================
- Hits          16643    16629      -14     
- Misses        12735    12744       +9     
- Partials       1551     1554       +3     
Impacted Files Coverage Δ
...in/java/org/apache/hive/beeline/KyuubiBeeLine.java 0.00% <ø> (ø)
...n/java/org/apache/hive/beeline/KyuubiCommands.java 0.00% <0.00%> (ø)
.../kyuubi/server/mysql/constant/MySQLErrorCode.scala 13.84% <0.00%> (-6.16%) ⬇️
...ache/kyuubi/server/mysql/MySQLCommandHandler.scala 77.77% <0.00%> (-4.05%) ⬇️
...ache/kyuubi/server/mysql/MySQLGenericPackets.scala 76.59% <0.00%> (-2.13%) ⬇️
...g/apache/kyuubi/operation/BatchJobSubmission.scala 75.82% <0.00%> (-1.65%) ⬇️
...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala 78.39% <0.00%> (-0.62%) ⬇️
...in/scala/org/apache/kyuubi/config/KyuubiConf.scala 97.46% <0.00%> (-0.07%) ⬇️

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

@pan3793 pan3793 added this to the v1.7.0 milestone Feb 24, 2023
@pan3793 pan3793 closed this in b39caed Feb 24, 2023
pan3793 pushed a commit that referenced this pull request Feb 24, 2023
…nt at the head …

…of beeline -e"

This reverts commit c489e29.

### _Why are the changes needed?_

#4333 (comment)

### _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.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #4406 from cfmcgrady/revert-4333.

Closes #4406

Closes #4305

c7fff72 [Fu Chen] Revert "[KYUUBI #4305][Bug] Backport HIVE-15820: comment at the head of beeline -e"

Authored-by: Fu Chen <cfmcgrady@gmail.com>
Signed-off-by: Cheng Pan <chengpan@apache.org>
(cherry picked from commit b39caed)
Signed-off-by: Cheng Pan <chengpan@apache.org>
@pan3793
Copy link
Member

pan3793 commented Feb 24, 2023

Thanks, merged to master/1.7

@cfmcgrady cfmcgrady deleted the revert-4333 branch June 6, 2023 10:52
turboFei added a commit that referenced this pull request Jun 16, 2023
…ne -e

### _Why are the changes needed?_

Backport apache/hive#1814

related kyuubi issues
#4305
#4333
#4406

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

- [ ] Add screenshots for manual tests if appropriate

- [x] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request

Closes #4972 from turboFei/beeline_head_command.

Closes #4305

04b235f [fwang12] [KYUUBI #4305][Bug] Backport HIVE-15820: comment at the head of beeline -e

Authored-by: fwang12 <fwang12@ebay.com>
Signed-off-by: fwang12 <fwang12@ebay.com>
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.

4 participants