Code review comment for lp:~vlad-lesin/percona-server/5.6-sql_timeout_twitter

Revision history for this message
Vlad Lesin (vlad-lesin) wrote :

> Maybe it isn't that bad to update test cases with new digests. I just
> wondering why that method of digest calculation has been chosen. Probably
> nobody should refer to digests in their scripts/applications/etc, and they are
> needed only for internal purposes.

It can lead to issues during merging. If several people make MP with updated tests merge captain should update that test results manually. But I think we can do nothing with it. We can't make rules for test scripts as we work with upstream code.

« Back to merge proposal