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

[improve][io] Upgrade Debezium to 1.9.7 #18465

Merged
merged 1 commit into from Nov 14, 2022
Merged

Conversation

dlg99
Copy link
Contributor

@dlg99 dlg99 commented Nov 14, 2022

Motivation

Upgrade debezium to 1.9.7 final to pick up latest fixes.

Modifications

Dependency upgrade

Verifying this change

  • Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Matching PR in forked repository

PR in forked repository: dlg99#5

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Nov 14, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 14, 2022

Codecov Report

Merging #18465 (c4cdc15) into master (dcf0b60) will decrease coverage by 17.62%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #18465       +/-   ##
=============================================
- Coverage     47.24%   29.62%   -17.63%     
+ Complexity    10440     5597     -4843     
=============================================
  Files           697      617       -80     
  Lines         67974    58473     -9501     
  Branches       7276     6090     -1186     
=============================================
- Hits          32116    17324    -14792     
- Misses        32284    38703     +6419     
+ Partials       3574     2446     -1128     
Flag Coverage Δ
unittests 29.62% <ø> (-17.63%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...n/java/org/apache/pulsar/client/api/RawReader.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pulsar/broker/admin/v1/Brokers.java 0.00% <0.00%> (-100.00%) ⬇️
...ava/org/apache/pulsar/broker/admin/v2/Brokers.java 0.00% <0.00%> (-100.00%) ⬇️
...va/org/apache/pulsar/broker/admin/v1/Clusters.java 0.00% <0.00%> (-100.00%) ⬇️
.../org/apache/pulsar/broker/admin/v1/Properties.java 0.00% <0.00%> (-100.00%) ⬇️
.../org/apache/pulsar/client/impl/RawMessageImpl.java 0.00% <0.00%> (-100.00%) ⬇️
.../apache/pulsar/broker/admin/v2/ResourceGroups.java 0.00% <0.00%> (-100.00%) ⬇️
...ar/common/naming/PartitionedManagedLedgerInfo.java 0.00% <0.00%> (-100.00%) ⬇️
...e/pulsar/broker/admin/impl/ResourceQuotasBase.java 0.00% <0.00%> (-96.43%) ⬇️
...he/pulsar/broker/service/AnalyzeBacklogResult.java 0.00% <0.00%> (-92.31%) ⬇️
... and 247 more

Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

dlg99 added a commit to datastax/pulsar that referenced this pull request Nov 14, 2022
@nicoloboschi nicoloboschi merged commit c47f2ed into apache:master Nov 14, 2022
@dlg99 dlg99 deleted the dbz_1.9.7 branch November 28, 2022 05:21
@lhotari lhotari added this to the 2.12.0 milestone Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants