[GitHub] [drill] cgivre commented on a change in pull request #2122: [DRILL-7817] Allow immediate parsing of the useragent from within the HTTPD LogFormat format plugin.

[GitBox <gi...@apache.org>]

cgivre commented on a change in pull request #2122:
URL: https://github.com/apache/drill/pull/2122#discussion_r537838475



##########
File path: contrib/format-httpd/README.md
##########
@@ -10,15 +10,16 @@ There are five fields which you can to configure in order for Drill to read web
 * **`extensions`**:  The file extension of your web server logs.  Defaults to `httpd`.
 * **`maxErrors`**:  Sets the plugin error tolerance. When set to any value less than `0`, Drill will ignore all errors. If unspecified then maxErrors is 0 which will cause the query to fail on the first error.
 * **`flattenWildcards`**: There are a few variables which Drill extracts into maps.  Defaults to `false`.
-
+* **`enableUserAgentParser`**:  This enables the additional Yauaa useragent parser. If unspecified this is disabled because there is a noticeable startup overhead of this plugin, even if not used.
 

Review comment:
       I wonder if we could do something here to only activate this function if:
   1.  The user executes a star query OR
   2.  They specified a field with `user-agent` in the field name?
   
   Do you think that would work?  
   
   




----------------------------------------------------------------
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:
users@infra.apache.org