jakartaee / jakartaee/messaging

Change XAConnectionFactory to extend ConnectionFactory

Open
#71 5 comments 0 reactions 0 assignees View on GitHub
pd20-forreview-minor Priority: Major Type: Improvement
Dominant language
Java
Stars
49
Forks
34
PR merge metrics
No merged PRs in 30d

Description

JMS 1.1, includes the following three interface definitions:

```
public interface XATopicConnectionFactory extends XAConnectionFactory, TopicConnectionFactory

public interface XAQueueConnectionFactory extends XAConnectionFactory, QueueConnectionFactory

public interface XAConnectionFactory
```

why does XAConnectionFactory not extend ConnectionFactory? This is inconsistent.

Please consider changing XAConnectionFactory to extend ConnectionFactory.
#### Affected Versions
[1.1]

Contributor guide

Open the contributing guide

Research direction

Start by locating the Java declarations for XAConnectionFactory, XATopicConnectionFactory, XAQueueConnectionFactory, and ConnectionFactory in this repository, then compare them with the JMS 1.1 definitions quoted in the issue. Review the compatibility implications of the proposed inheritance and existing tests; done means the interfaces consistently reflect the intended relationship and the relevant checks pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
api
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.