Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bug Report: Exception When Datasource Isolationlevel 'TRANSACTION_READ_UNCOMMITTED' is Used / FISH-6472 #5885

Open
fschuerer opened this issue Aug 10, 2022 · 2 comments
Assignees
Labels
Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev Type: Bug Label issue as a bug defect

Comments

@fschuerer
Copy link

Description


Hello,

when using the isolationlevel 'TRANSACTION_READ_UNCOMMITTED' in 'DataSourceDefinition' an exception is thrown

Current Outcome

Caused by: javax.resource.ResourceException: Invalid transaction isolation; the transaction isolation level can be empty or any of the following: read-uncommitted, read-committed, repeatable-read, serializable, custom

Steps to reproduce

@ApplicationScoped
@DataSourceDefinition(
        name = "java:app/jdbc/db",
        className = "org.postgresql.xa.PGXADataSource",
        serverName = "${MPCONFIG=POSTGRES_SERVER_NAME:localhost}",
        databaseName = "${MPCONFIG=POSTGRES_DB:postgres}",
        user = "${MPCONFIG=POSTGRES_USER:postgres}",
        password = "${MPCONFIG=POSTGRES_PASSWORD}",
        isolationLevel = Connection.TRANSACTION_READ_UNCOMMITTED, 
        portNumber = 5432
)
public class DatabaseConfiguration {
}

Environment

  • Distribution: Server Full Profile 5.2022.2
  • JDK Version: 11 - OpenJDK
  • Operating System: Windows

Analysis

In 'com.sun.appserv.connectors.internal.api.ConnectorsUtil' at line 542 is missing a 't':

image

@fschuerer fschuerer added Status: Open Issue has been triaged by the front-line engineers and is being worked on verification Type: Bug Label issue as a bug defect labels Aug 10, 2022
@JamesHillyard JamesHillyard self-assigned this Aug 10, 2022
@JamesHillyard JamesHillyard added Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev and removed Status: Open Issue has been triaged by the front-line engineers and is being worked on verification labels Aug 10, 2022
@JamesHillyard
Copy link
Member

Hi @fschuerer,

Thank you for reporting this issue, I have been able to successfully reproduce it. I have raised an internal issue FISH-6472 to address this bug. We always encourage you to contribute, so if you wish you can submit a PR with a fix and we will gladly review it.

As you have already identified the fix, I would strongly recommend you submit a PR for this to get the fix as quick as possible. The following documentation pages may help you in doing so:

If you choose not to submit a PR, we will pick up the issue at some point in the future.

Thanks,
James

@JamesHillyard JamesHillyard changed the title Bug Report: Exception when datasource isolationlevel 'TRANSACTION_READ_UNCOMMITTED' is used Bug Report: Exception When Datasource Isolationlevel 'TRANSACTION_READ_UNCOMMITTED' is Used / FISH-6472 Aug 11, 2022
@myresd
Copy link
Contributor

myresd commented Jan 29, 2024

Hi @JamesHillyard,

I ran into the same issue so I opened a PR here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Status: Accepted Confirmed defect or accepted improvement to implement, issue has been escalated to Platform Dev Type: Bug Label issue as a bug defect
Projects
None yet
Development

No branches or pull requests

3 participants