Barry Pollard
Barry Pollard
FYI @GJFR @vikvanderlinden @JannisBush On the first, this is noted in the text; > However, the increasing adoption of the nonce- and strict-dynamic keywords is a positive development. By using...
> OT: shouldn't it be Percentage and not Percent? IMHO yes. But I think this is a European versus US English thing and I'm European. Percentage is more common in...
`TABLESAMPLE SYSTEM (0.001 PERCENT)` says only run this on 0.001% of the rows in the table. This is good during query development as faster (and cheaper!) to run than on...
Fixed in #3863
`TABLESAMPLE` is used to restrict rows for quicker testing. The submitted queries shouldn't use this as then it won't be the full dataset.
Looks like this is included in #3791
There are two spaces between these two terms: `'JSS', 'React Router'` which the linter doesn't like. It (quite rightly!) thinks one space is enough. You can tell this btw because...
Actually, we're looking to convert these queries to the new data model. I've done that for you if you could add to the branch? Here's the first query: ```sql #standardSQL...
And here's the second: ```sql #standardSQL CREATE TEMP FUNCTION getFuguAPIs(data STRING) RETURNS ARRAY LANGUAGE js AS ''' const $ = JSON.parse(data); return Object.keys($); '''; SELECT client, page, COUNT(DISTINCT fuguAPI) AS...
I made some tweaks to handle secondary pages better in https://github.com/HTTPArchive/almanac.httparchive.org/pull/3747/commits/af0f96a1bb77926f1bab338bccb100dd9d704f04