trafficserver icon indicating copy to clipboard operation
trafficserver copied to clipboard

Add log field names pqu* and deprecates cqu*

Open maskit opened this issue 3 years ago • 1 comments

This is for #8196, and the target branch is 10-Dev because it deprecates a couple of log fields.

#8015 corrected errors on the documentation, but the field names were kept as they are for compatibility. This PR deprecates the incorrectly named fields, cpu, cquc, cqup, and cqus. And introduces pqu, pquc, pqup, and pqus as replacements.

I know variable names used in Log.cc still has "client_request" in their names and those should be renamed accordingly, but I wanted to make this field name change before it's too late since we can make this change only on a new major version release.

maskit avatar Aug 10 '22 03:08 maskit

I had to update a binary test data, and I made the change like this.

sed -e 's/cquc/pquc/' src/traffic_logstats/tests/logstats.blog.orig > src/traffic_logstats/tests/logstats.blog

maskit avatar Aug 10 '22 06:08 maskit