IGNITE-13315, please review the pull request

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

IGNITE-13315, please review the pull request

Konstantin Sirotkin
Hello!

Please review the changes.
This PR contains one line:)

Thanks.

---
Kind Regards,
Konstantin Sirotkin

Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-13315, please review the pull request

dmagda
Konstantin, thanks a lot for the contribution! I've reviewed and merged the
improvement.

Please do us a favor and:

   - In addition to the ticket number include its name in the email
   subject. Otherwise, most of the committers won't pay attention to the email
   and simply pass by. For instance, this name "Review needed:
   IGNITE-13315, Uninformative TxDumpsThrottling toString output" should have
   caught attention of more committers.
   - Include a link to the ticket in the body of the email so that
   reviewers can open it really quickly.

Look forward to more contributions from you! Keep it up.

-
Denis


On Fri, Jul 31, 2020 at 9:14 AM Konstantin Sirotkin <
[hidden email]> wrote:

> Hello!
>
> Please review the changes.
> This PR contains one line:)
>
> Thanks.
>
> ---
> Kind Regards,
> Konstantin Sirotkin
>
>