[Mod_log_sql] Time of Request

Marius Karthaus Marius at Karthaus.nl
Tue Feb 15 13:02:09 EST 2005


Will this be part of the main tree any time soon ?



Shawn Michael wrote:

> That's what apache provides by default.  Starting with apache 2.0 you 
> do have the option of logging microseconds.  Back on Jan 25th I 
> submitted a patch to the list that enables this option.  I have 
> attached the patch here also.
>
> Marius Karthaus wrote:
>
>> Hi,
>> Why is the 'T' (time of processing) defined as a smallint?
>> Most requests end up as 0 or 1 not much info supplied like that.
>> Whouldn't it be better to have it in microseconds?
>>
>> Greets,
>> Marius Karthaus
>> LowVoice
>>
>> _______________________________________________
>> Download the latest version at 
>> http://www.outoforder.cc/projects/apache/mod_log_sql/
>>
>> To unsubscribe send an e-mail to 
>> mod_log_sql-unsubscribe at lists.outoforder.cc
>>
>
>------------------------------------------------------------------------
>
>diff -x .svn --recursive --unified mod_log_sql/contrib/create_tables.sql mod_log_sql.new/contrib/create_tables.sql
>--- mod_log_sql/contrib/create_tables.sql	2004-11-24 15:34:22.000000000 -0700
>+++ mod_log_sql.new/contrib/create_tables.sql	2004-11-24 15:16:04.000000000 -0700
>@@ -23,7 +23,8 @@
> 	ssl_maxkeysize smallint unsigned,
> 	status smallint unsigned ,
> 	time_stamp int unsigned ,
>-	virtual_host varchar(255)
>+	virtual_host varchar(255) ,
>+    request_duration_ms int unsigned
> );
> 
> create table notes (
>diff -x .svn --recursive --unified mod_log_sql/functions20.h mod_log_sql.new/functions20.h
>--- mod_log_sql/functions20.h	2004-11-24 15:34:24.000000000 -0700
>+++ mod_log_sql.new/functions20.h	2004-11-24 15:06:12.000000000 -0700
>@@ -96,6 +96,14 @@
> 	return apr_psprintf(r->pool, "%" APR_TIME_T_FMT, apr_time_sec(duration));
> }
> 
>+
>+static const char *extract_request_duration_microseconds(request_rec *r, char *a)
>+{
>+    return apr_psprintf(r->pool, "%" APR_TIME_T_FMT,
>+                        (apr_time_now() - r->request_time));
>+}
>+
>+
> static const char *extract_request_timestamp(request_rec *r, char *a)
> {
> 	return apr_psprintf(r->pool, "%"APR_TIME_T_FMT, apr_time_sec(apr_time_now()));
>diff -x .svn --recursive --unified mod_log_sql/mod_log_sql.c mod_log_sql.new/mod_log_sql.c
>--- mod_log_sql/mod_log_sql.c	2004-11-24 15:34:24.000000000 -0700
>+++ mod_log_sql.new/mod_log_sql.c	2004-11-24 15:09:55.000000000 -0700
>@@ -490,6 +490,9 @@
>     log_sql_register_item(s,p,'U', extract_request_uri,       "request_uri",      1, 1);
>     log_sql_register_item(s,p,'v', extract_virtual_host,      "virtual_host",     0, 1);
>     log_sql_register_item(s,p,'V', extract_server_name,       "virtual_host",     0, 1);
>+#if defined(WITH_APACHE20)
>+    log_sql_register_item(s,p,'D', extract_request_duration_microseconds,  "request_duration_ms", 1, 0);
>+#endif
> 
>     if (global_config.announce) {
>         ap_add_version_component(p, PACKAGE_NAME"/"PACKAGE_VERSION);
>diff -x .svn --recursive --unified mod_log_sql/mod_log_sql_dbi.c mod_log_sql.new/mod_log_sql_dbi.c
>--- mod_log_sql/mod_log_sql_dbi.c	2004-11-24 15:34:24.000000000 -0700
>+++ mod_log_sql.new/mod_log_sql_dbi.c	2004-11-24 15:13:37.000000000 -0700
>@@ -180,7 +180,8 @@
>        ssl_maxkeysize smallint unsigned,\
>        status smallint unsigned,\
>        time_stamp int unsigned,\
>-       virtual_host varchar(255))";
>+       virtual_host varchar(255),\
>+       request_duration_ms int unsigned)";
> 		break;
> 	case LOGSQL_TABLE_COOKIES:
> 	case LOGSQL_TABLE_HEADERSIN:
>diff -x .svn --recursive --unified mod_log_sql/mod_log_sql_mysql.c mod_log_sql.new/mod_log_sql_mysql.c
>--- mod_log_sql/mod_log_sql_mysql.c	2004-11-24 15:34:24.000000000 -0700
>+++ mod_log_sql.new/mod_log_sql_mysql.c	2004-11-24 15:37:52.000000000 -0700
>@@ -210,7 +210,8 @@
>        ssl_maxkeysize smallint unsigned,\
>        status smallint unsigned,\
>        time_stamp int unsigned,\
>-       virtual_host varchar(255))";
>+       virtual_host varchar(255),\
>+       request_duration_ms int unsigned)";
> 		break;
> 	case LOGSQL_TABLE_COOKIES:
> 	case LOGSQL_TABLE_HEADERSIN:
>diff -x .svn --recursive --unified mod_log_sql/mod_log_sql_pgsql.c mod_log_sql.new/mod_log_sql_pgsql.c
>--- mod_log_sql/mod_log_sql_pgsql.c	2004-11-24 15:34:24.000000000 -0700
>+++ mod_log_sql.new/mod_log_sql_pgsql.c	2004-11-24 15:14:49.000000000 -0700
>@@ -195,7 +195,8 @@
>        ssl_maxkeysize smallint unsigned,\
>        status smallint unsigned,\
>        time_stamp int unsigned,\
>-       virtual_host varchar(255))";
>+       virtual_host varchar(255),\
>+       request_duration_ms int unsigned)";
> 		break;
> 	case LOGSQL_TABLE_COOKIES:
> 	case LOGSQL_TABLE_HEADERSIN:
>  
>
>------------------------------------------------------------------------
>
>_______________________________________________
>Download the latest version at http://www.outoforder.cc/projects/apache/mod_log_sql/
>
>To unsubscribe send an e-mail to 
>mod_log_sql-unsubscribe at lists.outoforder.cc
>  
>




More information about the Mod_log_sql mailing list