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

[DEPS][TEST] Bump iceberg 0.12.0 #948

Closed
wants to merge 1 commit into from
Closed

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Aug 17, 2021

Why are the changes needed?

Close #943

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

@codecov-commenter
Copy link

Codecov Report

Merging #948 (47a8a65) into master (f21e118) will increase coverage by 0.72%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #948      +/-   ##
============================================
+ Coverage     79.10%   79.83%   +0.72%     
  Complexity       11       11              
============================================
  Files           146      146              
  Lines          5414     5414              
  Branches        654      654              
============================================
+ Hits           4283     4322      +39     
+ Misses          757      714      -43     
- Partials        374      378       +4     
Impacted Files Coverage Δ
...pache/kyuubi/sql/KyuubiQueryStagePreparation.scala 79.41% <0.00%> (-0.99%) ⬇️
...org/apache/kyuubi/ha/client/ServiceDiscovery.scala 44.62% <0.00%> (+0.82%) ⬆️
...ache/kyuubi/operation/KyuubiOperationManager.scala 64.15% <0.00%> (+5.66%) ⬆️
.../apache/kyuubi/client/KyuubiSyncThriftClient.scala 81.81% <0.00%> (+9.09%) ⬆️
...he/kyuubi/engine/spark/shim/CatalogShim_v3_0.scala 88.15% <0.00%> (+26.31%) ⬆️
...scala/org/apache/kyuubi/operation/GetColumns.scala 88.88% <0.00%> (+88.88%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f21e118...47a8a65. Read the comment docs.

@pan3793 pan3793 self-assigned this Aug 18, 2021
@pan3793 pan3793 added this to the v1.3.0 milestone Aug 18, 2021
@pan3793 pan3793 requested a review from yaooqinn August 18, 2021 02:19
@pan3793 pan3793 changed the title [BUILD] Bump iceberg 0.12.0 [DEPS][TEST] Bump iceberg 0.12.0 Aug 18, 2021
@yaooqinn yaooqinn closed this in b88d9b2 Aug 18, 2021
@yaooqinn
Copy link
Member

thanks, merged to master / v1.3

yaooqinn pushed a commit that referenced this pull request Aug 18, 2021
<!--
Thanks for sending a pull request!

Here are some tips for you:
  1. If this is your first time, please read our contributor guidelines: https://kyuubi.readthedocs.io/en/latest/community/contributions.html
  2. If the PR is related to an issue in https://github.com/apache/incubator-kyuubi/issues, add '[KYUUBI #XXXX]' in your PR title, e.g., '[KYUUBI #XXXX] Your PR title ...'.
  3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., '[WIP][KYUUBI #XXXX] Your PR title ...'.
-->

### _Why are the changes needed?_
<!--
Please clarify why the changes are needed. For instance,
  1. If you add a feature, you can talk about the use case of it.
  2. If you fix a bug, you can clarify why it is a bug.
-->
Close #943

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

Closes #948 from pan3793/iceberg.

Closes #948

47a8a65 [Cheng Pan] Bump iceberg 0.12.0

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Kent Yao <yao@apache.org>
(cherry picked from commit b88d9b2)
Signed-off-by: Kent Yao <yao@apache.org>
@pan3793 pan3793 deleted the iceberg branch August 18, 2021 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bump Iceberg 0.12.0
3 participants