jonkeane commented on pull request #10232:
URL: https://github.com/apache/arrow/pull/10232#issuecomment-831565905


   The changes to tasks.yml looks good, however a few notes: 
   
   * Could you rebase/squash your commits here so that we don't have all of 
that extraneous history? It should be fine when we squash + merge it, but we 
want to make sure we're being clean with this and it's not touching other 
things accidentally / unnecessarily
   * Dig more into the failure that you noted. What failed? Does it make sense 
that these changes are touching that? 
   * Could you update the format of the PR to match our style (pick either of 
the two tickets we have for this to use as the ticket number + mark the other 
as a duplicate) 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to