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
ZOOKEEPER-3722: make logs of ResponseCache more readable #1253
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
|
||
getChildrenResponseCache = new ResponseCache(Integer.getInteger( | ||
GET_CHILDREN_RESPONSE_CACHE_SIZE, | ||
ResponseCache.DEFAULT_RESPONSE_CACHE_SIZE)); | ||
ResponseCache.DEFAULT_RESPONSE_CACHE_SIZE), "getChild"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's replace "getChild" with "getChildren" so it's consistent with the op code and the cache name getChildrenResponseCache
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
replaced "getChild" with "getChildren".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for changing the names. i'll merge this once all tests are green.
As per [ZOOKEEPER-3722](https://issues.apache.org/jira/browse/ZOOKEEPER-3722), we need to make ResponseCache logs more readable by adding the request type to the logs. For this I made the following changes: - Added a parameter named requestType to ResponseCache constructor. - Passed requestType as getData and getChild when the constructor is called to initialize readResponseCache and getChildrenResponseCache respectively. Please let me know if additional changes are required. Author: Nishanth Entoor <entoor.nishanth@gmail.com> Reviewers: hanm, HorizonNet, maoling Closes apache#1253 from nishanth-entoor/ZOOKEEPER-3722
As per [ZOOKEEPER-3722](https://issues.apache.org/jira/browse/ZOOKEEPER-3722), we need to make ResponseCache logs more readable by adding the request type to the logs. For this I made the following changes: - Added a parameter named requestType to ResponseCache constructor. - Passed requestType as getData and getChild when the constructor is called to initialize readResponseCache and getChildrenResponseCache respectively. Please let me know if additional changes are required. Author: Nishanth Entoor <entoor.nishanth@gmail.com> Reviewers: hanm, HorizonNet, maoling Closes apache#1253 from nishanth-entoor/ZOOKEEPER-3722
As per [ZOOKEEPER-3722](https://issues.apache.org/jira/browse/ZOOKEEPER-3722), we need to make ResponseCache logs more readable by adding the request type to the logs. For this I made the following changes: - Added a parameter named requestType to ResponseCache constructor. - Passed requestType as getData and getChild when the constructor is called to initialize readResponseCache and getChildrenResponseCache respectively. Please let me know if additional changes are required. Author: Nishanth Entoor <entoor.nishanth@gmail.com> Reviewers: hanm, HorizonNet, maoling Closes apache#1253 from nishanth-entoor/ZOOKEEPER-3722
As per [ZOOKEEPER-3722](https://issues.apache.org/jira/browse/ZOOKEEPER-3722), we need to make ResponseCache logs more readable by adding the request type to the logs. For this I made the following changes: - Added a parameter named requestType to ResponseCache constructor. - Passed requestType as getData and getChild when the constructor is called to initialize readResponseCache and getChildrenResponseCache respectively. Please let me know if additional changes are required. Author: Nishanth Entoor <entoor.nishanth@gmail.com> Reviewers: hanm, HorizonNet, maoling Closes apache#1253 from nishanth-entoor/ZOOKEEPER-3722
As per ZOOKEEPER-3722, we need to make ResponseCache logs more readable by adding the request type to the logs. For this I made the following changes:
Please let me know if additional changes are required.