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.

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

Transaction timeout limit hints #203

Open
brack opened this issue Sep 28, 2021 · 0 comments
Open

Transaction timeout limit hints #203

brack opened this issue Sep 28, 2021 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@brack
Copy link
Contributor

brack commented Sep 28, 2021

I am putting myself in the shoes of someone trying to experiment with a new, more complex transaction. With that in mind, I start a transaction and start shaping my transaction. From a QLDB perspective, I should probably be experimenting outside of a transaction, and then having something ready to go to work within the (current) 30 second limit, but I might be coming from not having to think about this.

As I'm querying, I get an error communication failure: InvalidSessionException: Transaction 05K97LfBBVa60AK0qMbc6R has expired

This could be a little jarring - did I do something wrong? Do I need to update the expiration time? is QLDB not working? etc? I might not realize that it's a hard limit of the service at this point.

If feasible, I would suggest a couple small tweaks to explicitly describe the issue so there's no confusion as to whether it's user error or QLDB error.

Suggestions:

1/ when a transaction is started, print out the transaction time limit. Even better if we could have a countdown like we are thinking for the console query editor.
2/ when the error is received, would it be worth also stating whatever the transaction time limit is?

At the very least, perhaps we should consider updating the error message to say "Transaction ... has expired, please see QLDB limits documentation for more information." This way, it is immediately clear they're bumping against a service limit rather than thinking they're not doing something correctly.

Tangential: we should consider doing similar for session expiration errors as well, if they hit the 13-17 minute session timeout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants