sergeykolychev commented on issue #10460: MXNET-295 [Perl] Added support for 
linalg operators.
URL: https://github.com/apache/incubator-mxnet/pull/10460#issuecomment-379821889
 
 
   Got it, thank you Mark. I'll make a rule to wait at least 72 hours before
   merging (approval from perl contributors and tests pass assumed as well).
   
   On Mon, Apr 9, 2018 at 9:44 AM, Marco de Abreu <notificati...@github.com>
   wrote:
   
   > I think @nswamy <https://github.com/nswamy> knows Perl, right?
   >
   > I definitely see your point and appreciate that you don't want to bother
   > others, but I don't think a bit of peer-reviewing is going to totally fine
   > - even if it's being by people who are not familiar with the language
   > itself. If there's really no way to get somebody to verify your PR, I'd
   > suggest the Apache approach about their rules for lazy-votes. If nobody
   > objects within 72h hours, you're free to move on without further approval.
   >
   > Please don't misunderstand me, this is no criticism about your skills or
   > anything, I just think we should give some time to the community to have a
   > quick look and point out possible concerns our users and developers have.
   > Of course, you always have the liberty to merge critical PRs yourself
   > immediately.
   >
   > —
   > You are receiving this because you were mentioned.
   > Reply to this email directly, view it on GitHub
   > 
<https://github.com/apache/incubator-mxnet/pull/10460#issuecomment-379817355>,
   > or mute the thread
   > 
<https://github.com/notifications/unsubscribe-auth/AYSk2E4hRfjhqeRygRgYsGKnS0_6duPEks5tm5AIgaJpZM4TLtC->
   > .
   >
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to