* the patch adds a new file named response_time_distribution.patch.update. what is that for?
* we are going to move tests to the corresponding patches. so please change it so that all tests code to response_time_distribution.patch rather than a separate directory
* percona_query_response_time_show.inc: "\ No newline at end of file"
* percona_query_response_time_flush.inc: "\ No newline at end of file"
* too much conditional code in .inc files. It would actually be more simple and readable to move the _common_ code into .inc file(s) and let replication- and SP-related tests contain the rest.
* since most of the actual code changes occur in the response_time_distribution.patch.update which was apparently a mistake, I would like the above issues to be addressed before proceeding with review.
* the patch adds a new file named response_ time_distributi on.patch. update. what is that for?
* we are going to move tests to the corresponding patches. so please change it so that all tests code to response_ time_distributi on.patch rather than a separate directory
* percona_ query_response_ time_show. inc: "\ No newline at end of file"
* percona_ query_response_ time_flush. inc: "\ No newline at end of file"
* too much conditional code in .inc files. It would actually be more simple and readable to move the _common_ code into .inc file(s) and let replication- and SP-related tests contain the rest.
* since most of the actual code changes occur in the response_ time_distributi on.patch. update which was apparently a mistake, I would like the above issues to be addressed before proceeding with review.