cgivre opened a new pull request #1635: DRILL-7021: HTTPD Throws NPE and Doesn't Recognize Timeformat URL: https://github.com/apache/drill/pull/1635 This PR addresses several issues with the HTTPD format plugin. 1. Several fields were missing in the implementation which were causing a NPE if you included them explicitly in a query 2. The format plugin was not parsing time stamps correctly and projecting the fields as timestamps 3. The `request_user-agent` field had a dash in it which, if not escaped correctly, would cause problems in queries. 4. The unit tests were not satisfactory 5. The default format specified in the `bootstrap.json` file omitted the user agent string which most HTTPD logs do include. This PR fixes all these issues and updates the HTTPD log parser to version 5.2 which will accept Nginx logs.
---------------------------------------------------------------- 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