zookeeper-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [zookeeper] xoiss opened a new pull request #999: ZOOKEEPER-2891: Invalid processing of zookeeper_close for mutli-request
Date Sun, 23 Jun 2019 18:17:06 GMT
xoiss opened a new pull request #999: ZOOKEEPER-2891: Invalid processing of zookeeper_close
for mutli-request
URL: https://github.com/apache/zookeeper/pull/999
 
 
   When I call `zookeeper_close()` while there is a pending multi request, I expect the request
completes with `ZCLOSING` status.
   
   But with the existing code I actually get the following:
   * the program exits with `SIGABRT` from `assert(entry)` in `deserialize_multi()`
   * and even if I remove this assertion and just break the enclosing loop, the returned status
is `ZOK` but not `ZCLOSING`
   
   So, there are two defects with processing calls to `zookeeper_close()` for pending multi
requests: improper assertion in implementation and invalid status in confirmation.
   
   Proposed changes remove these defects.
   
   For more details see: https://issues.apache.org/jira/browse/ZOOKEEPER-2891

----------------------------------------------------------------
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


With regards,
Apache Git Services

Mime
View raw message