cgivre commented on issue #1973: Fix typo in class name of FormSecurityHandler
URL: https://github.com/apache/drill/pull/1973#issuecomment-583792630
 
 
   Thanks for the contribution!  3 things:
   1.  Could you please uncomment or remove the logger line?
   2.  Drill PRs must have a corresponding JIRA issue.  Even though this is a 
minor fix, would you mind please creating a JIRA issue and adding that to the 
title of the PR?
   3.  Please add a blank line to the end of this file.
   
   Other than that, LGTM.

----------------------------------------------------------------
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]


With regards,
Apache Git Services

Reply via email to