On Tue, Oct 18, 2016 at 09:31:25AM +0800, Qu Wenruo wrote:
> The patchset does the following things:
> 1) Enhance comment for qgroup, rename 2 functions
>Explain the how qgroup works, so new developers won't waste too much
>time digging into the boring codes.
>
>The qgroup work flow is
On Fri, Oct 28, 2016 at 08:33:34AM +0800, Qu Wenruo wrote:
> Any comment?
>
> Especially the final patch will fix a long standing bug.
>From my perspective, review is required (but I appreciate Goldwyn's
testing). Who's going to do that, I don't know.
--
To unsubscribe from this list: send the li
On 10/27/2016 07:33 PM, Qu Wenruo wrote:
> Any comment?
>
> Especially the final patch will fix a long standing bug.
>
While I have tested it, and it works, I did not get time to review it.
So, you can have my
Tested-by: Goldwyn Rodrigues
> Thanks,
> Qu
>
> At 10/18/2016 09:31 AM, Qu Wenru
Any comment?
Especially the final patch will fix a long standing bug.
Thanks,
Qu
At 10/18/2016 09:31 AM, Qu Wenruo wrote:
The patchset does the following things:
1) Enhance comment for qgroup, rename 2 functions
Explain the how qgroup works, so new developers won't waste too much
time di
The patchset does the following things:
1) Enhance comment for qgroup, rename 2 functions
Explain the how qgroup works, so new developers won't waste too much
time digging into the boring codes.
The qgroup work flow is split into 3 main phrases:
Reverse, Trace, Account.
And rename f