Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

[transactions] Implement KIP-664 - DESCRIBE_TRANSACTIONS #1985

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

gaoran10
Copy link
Contributor

This PR needs #1984.

Motivation

Implement KIP-664 - DESCRIBE_TRANSACTIONS.

Modifications

Handle the command DESCRIBE_TRANSACTIONS.

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@gaoran10 gaoran10 self-assigned this Jul 30, 2023
@gaoran10 gaoran10 marked this pull request as draft July 30, 2023 05:35
@github-actions github-actions bot added the no-need-doc This pr does not need any document label Jul 30, 2023
@github-actions
Copy link

@gaoran10:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

1 similar comment
@github-actions
Copy link

@gaoran10:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions bot added the doc-info-missing This pr needs to mark a document option in description label Jul 30, 2023
@github-actions
Copy link

@gaoran10:Thanks for your contribution. For this PR, do we need to update docs?
(The PR template contains info about doc, which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)

@github-actions github-actions bot added the doc-info-missing This pr needs to mark a document option in description label Jul 30, 2023
@codecov
Copy link

codecov bot commented Jul 30, 2023

Codecov Report

Merging #1985 (6108953) into master (26980a3) will decrease coverage by 0.15%.
Report is 1 commits behind head on master.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1985      +/-   ##
============================================
- Coverage     17.05%   16.91%   -0.15%     
  Complexity      728      728              
============================================
  Files           191      191              
  Lines         14235    14357     +122     
  Branches       1332     1351      +19     
============================================
  Hits           2428     2428              
- Misses        11631    11753     +122     
  Partials        176      176              
Files Changed Coverage Δ
...ative/pulsar/handlers/kop/KafkaCommandDecoder.java 0.30% <0.00%> (-0.01%) ⬇️
...ative/pulsar/handlers/kop/KafkaRequestHandler.java 0.99% <0.00%> (-0.01%) ⬇️
...e/pulsar/handlers/kop/KafkaTopicLookupService.java 1.92% <0.00%> (-0.04%) ⬇️
...ndlers/kop/coordinator/group/GroupCoordinator.java 0.00% <0.00%> (ø)
...oordinator/transaction/TransactionCoordinator.java 0.00% <0.00%> (ø)
.../kop/coordinator/transaction/TransactionState.java 0.00% <0.00%> (ø)
...ordinator/transaction/TransactionStateManager.java 0.00% <0.00%> (ø)
.../pulsar/handlers/kop/utils/KafkaResponseUtils.java 0.00% <0.00%> (ø)

@gaoran10 gaoran10 force-pushed the support_describe_transactions branch from c9e90c2 to 6108953 Compare August 9, 2023 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
doc-info-missing This pr needs to mark a document option in description release/2.10.4 release/2.11 release/3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants