Skip to content

Increase Refactoring Request Timeout#11278

Merged
tsmaeder merged 1 commit intoeclipse-che:5730_java_ls_pocfrom
tsmaeder:11275_up_refactoring_timeout
Sep 19, 2018
Merged

Increase Refactoring Request Timeout#11278
tsmaeder merged 1 commit intoeclipse-che:5730_java_ls_pocfrom
tsmaeder:11275_up_refactoring_timeout

Conversation

@tsmaeder
Copy link
Copy Markdown
Contributor

What does this PR do?

It increases the request timeout on move and rename refactoring calls to 30 seconds. Since refactorings are large operations, it makes sense to have a different timeout than "hover", for example

What issues does this PR fix or reference?

Timeout on Refactorings in Selenium Tests #11275

@tsmaeder tsmaeder added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. target/branch Indicates that a PR will be merged into a branch other than master. e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. team/languages labels Sep 19, 2018
@tsmaeder tsmaeder requested review from svor and tolusha September 19, 2018 08:29
Signed-off-by: Thomas Mäder <tmader@redhat.com>
@tsmaeder tsmaeder force-pushed the 11275_up_refactoring_timeout branch from 7b99d58 to 4465548 Compare September 19, 2018 15:10
@tsmaeder tsmaeder merged commit 4df82ff into eclipse-che:5730_java_ls_poc Sep 19, 2018
@benoitf benoitf removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Sep 19, 2018
tsmaeder added a commit that referenced this pull request Sep 20, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Sep 26, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 1, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 5, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 12, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 16, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
tsmaeder added a commit that referenced this pull request Oct 17, 2018
Signed-off-by: Thomas Mäder <tmader@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e-test/failure Issues that is related to a test failures reported by our CI platform and our QE. target/branch Indicates that a PR will be merged into a branch other than master.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants