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.

Sign up for GitHub

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

Jump to bottom

manage cookies for Transaction.readStatus() #989

Closed
ehennum opened this issue Aug 15, 2018 · 3 comments
Closed

manage cookies for Transaction.readStatus() #989

ehennum opened this issue Aug 15, 2018 · 3 comments
Labels
Bug closed major
Milestone
java-client-api-4...

Comments

@ehennum
Copy link
Contributor

ehennum commented Aug 15, 2018

The Transaction.readStatus() method does not work with a load balancer because it doesn't call addTransactionScopedCookies() to manage the cookies used for host affinity.

Tracked as server bug 50603
@ehennum ehennum self-assigned this Aug 15, 2018
@ehennum ehennum added this to the java-client-api-4.1.1 milestone Aug 15, 2018
ehennum pushed a commit that referenced this issue Aug 16, 2018
manage cookies as part of transaction status request #989
08b4b54
@ehennum ehennum assigned georgeajit and unassigned ehennum Aug 16, 2018
@ehennum ehennum added test and removed minor labels Aug 16, 2018
@ehennum
Copy link
Contributor Author

ehennum commented Aug 16, 2018

Should cause no regressions.

After ELB configuration to use hostId, GenericDocumentTest.testCommit() should run to completion successfully.

@ehennum
Copy link
Contributor Author

ehennum commented Aug 17, 2018

With help from Matt and Hongyang, I confirmed that GenericDocumentTest.testCommit() now runs to completion successfully for an ELB configured for HostId.

ehennum pushed a commit that referenced this issue Aug 23, 2018
manage cookies as part of transaction status request #989
bf0ce65 
(cherry picked from commit 08b4b54)
@georgeajit georgeajit added ship and removed test labels Oct 3, 2018
@georgeajit
Copy link
Contributor

Tested by Srinath. Will be testing one final round.

@ehennum ehennum added closed and removed ship labels Jul 22, 2020
@ehennum ehennum added major and removed fix labels Oct 12, 2020
@ehennum ehennum closed this as completed Oct 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug closed major
Projects
None yet
Milestone
java-client-api-4.1.1
Development

No branches or pull requests
3 participants
@jmakeig @ehennum @georgeajit