You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2020/09/17 14:33:32 UTC
[GitHub] [geode-native] alb3rtobr opened a new pull request #651: Remove bucketServerLocation if timeout error
alb3rtobr opened a new pull request #651:
URL: https://github.com/apache/geode-native/pull/651
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [geode-native] alb3rtobr commented on pull request #651: Remove bucketServerLocation if timeout error
Posted by GitBox <gi...@apache.org>.
alb3rtobr commented on pull request #651:
URL: https://github.com/apache/geode-native/pull/651#issuecomment-701974141
I will use a separated ticket and branch for this issue, so im closing this PR. New one is here https://github.com/apache/geode-native/pull/662
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [geode-native] alb3rtobr commented on pull request #651: Remove bucketServerLocation if timeout error
Posted by GitBox <gi...@apache.org>.
alb3rtobr commented on pull request #651:
URL: https://github.com/apache/geode-native/pull/651#issuecomment-698811497
@pdxcodemonkey @echobravopapa Have you had the opportunity to check how could I add a test to verify this?
What if I create a separate ticket for the test case and this change is merge in the meanwhile? As after a timeout there is always a IO error, this change seems an improvement in the current situation when that happens, as it is in our case.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [geode-native] alb3rtobr commented on pull request #651: Remove bucketServerLocation if timeout error
Posted by GitBox <gi...@apache.org>.
alb3rtobr commented on pull request #651:
URL: https://github.com/apache/geode-native/pull/651#issuecomment-698811497
@pdxcodemonkey @echobravopapa Have you had the opportunity to check how could I add a test to verify this?
What if I create a separate ticket for the test case and this change is merge in the meanwhile? As after a timeout there is always a IO error, this change seems an improvement in the current situation when that happens, as it is in our case.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [geode-native] alb3rtobr commented on pull request #651: Remove bucketServerLocation if timeout error
Posted by GitBox <gi...@apache.org>.
alb3rtobr commented on pull request #651:
URL: https://github.com/apache/geode-native/pull/651#issuecomment-696025816
@echobravopapa @pdxcodemonkey This is the PR with the change regarding the client metadata update when there is a timeout error that I mentioned in the dev list
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [geode-native] alb3rtobr commented on pull request #651: Remove bucketServerLocation if timeout error
Posted by GitBox <gi...@apache.org>.
alb3rtobr commented on pull request #651:
URL: https://github.com/apache/geode-native/pull/651#issuecomment-696025816
@echobravopapa @pdxcodemonkey This is the PR with the change regarding the client metadata update when there is a timeout error that I mentioned in the dev list
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [geode-native] alb3rtobr closed pull request #651: Remove bucketServerLocation if timeout error
Posted by GitBox <gi...@apache.org>.
alb3rtobr closed pull request #651:
URL: https://github.com/apache/geode-native/pull/651
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org