Skip to content

JENA-1078: Logging warning when transaction is ended without commiting#105

Merged
asfgit merged 2 commits intoapache:masterfrom
ajs6f:AddNoCommitWarning
Dec 9, 2015
Merged

JENA-1078: Logging warning when transaction is ended without commiting#105
asfgit merged 2 commits intoapache:masterfrom
ajs6f:AddNoCommitWarning

Conversation

@ajs6f
Copy link
Copy Markdown
Member

@ajs6f ajs6f commented Dec 8, 2015

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

begin(READ)-end() is a normal pattern. The code needs to know if it is considering a READ or WRITE transaction.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed with latest commit.

@asfgit asfgit merged commit 0a11e4a into apache:master Dec 9, 2015
asfgit pushed a commit that referenced this pull request Dec 9, 2015
@afs
Copy link
Copy Markdown
Member

afs commented Dec 11, 2015

This causes about 12 warnings from the test suite (Eclipse and maven).

Please do add tests and check the test suite runs. I will check them and fix or silence each one but it would be nice to be at least notified that it would happen.

@ajs6f
Copy link
Copy Markdown
Member Author

ajs6f commented Dec 11, 2015

I had no idea that throwing WARNINGs in the test suite was any kind of issue. ERRORs, yes, but not WARNINGs. That's a policy that might be recorded somewhere, perhaps here?

@ajs6f
Copy link
Copy Markdown
Member Author

ajs6f commented Dec 11, 2015

In any event, I can review them and eliminate them. No need for you to spend time doing that-- this is my code and I take responsibility for it. Would you reopen the ticket for me?

@afs
Copy link
Copy Markdown
Member

afs commented Dec 11, 2015

We have had the discussion about warnings again on the dev@ just recently.

I've already started on a review and some fixes. There are some other problems with abort/end. I will submit a PR here for detailed code review. The issues look harmless but that's because of the way the implementation of indexes happen to work. Different plugged in indexes would not be so safe.

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.

3 participants