IGNITE-5229 is ready for review

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

IGNITE-5229 is ready for review

Roman Shtykh
Igniters,
IGNITE-5229 is ready for review.Now Redis can have its default cache specified and used as a template when switching to other caches, as discussed earlier in this ml (please see the JIRA issue).

[1] https://issues.apache.org/jira/browse/IGNITE-5229
-- Roman
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-5229 is ready for review

Anton Vinogradov-2
Roman,

Today I added some tips how to request review in most efficient way.
See thread '*"Review workflow" changes to prevent "broken review" issues*'
for details.

I'd like to ask you to help me to check my theory that these tips works.

Could you please ask review using tips from
https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute#HowtoContribute-SubmittingforReview
?

Please let me know will tips help you or not.

On Wed, Jun 7, 2017 at 8:49 AM, Roman Shtykh <[hidden email]>
wrote:

> Igniters,
> IGNITE-5229 is ready for review.Now Redis can have its default cache
> specified and used as a template when switching to other caches, as
> discussed earlier in this ml (please see the JIRA issue).
>
> [1] https://issues.apache.org/jira/browse/IGNITE-5229
> -- Roman
>
Reply | Threaded
Open this post in threaded view
|

Re: IGNITE-5229 is ready for review

Roman Shtykh
Anton,
Sure. Actually, I do it pretty often when I see no progress on the ticket, and I think this approach is a good _option_ to raise some attention to the pull request.
-- Roman




    On Wednesday, June 7, 2017 8:38 PM, Anton Vinogradov <[hidden email]> wrote:
 

 Roman,

Today I added some tips how to request review in most efficient way.
See thread '*"Review workflow" changes to prevent "broken review" issues*'
for details.

I'd like to ask you to help me to check my theory that these tips works.

Could you please ask review using tips from
https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute#HowtoContribute-SubmittingforReview
?

Please let me know will tips help you or not.

On Wed, Jun 7, 2017 at 8:49 AM, Roman Shtykh <[hidden email]>
wrote:

> Igniters,
> IGNITE-5229 is ready for review.Now Redis can have its default cache
> specified and used as a template when switching to other caches, as
> discussed earlier in this ml (please see the JIRA issue).
>
> [1] https://issues.apache.org/jira/browse/IGNITE-5229
> -- Roman
>