Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
In boost_crawler_run() and boost_crawler_total_count() in the file boost.module, PostgreSQL (8.4.7) complains about the syntax of the SQL due to the column names in the SELECT not being present in the GROUP BY clause. An example of one of the errors is "ERROR: column "boost_crawler.id" must appear in the GROUP BY clause or be used in an aggregate function at character 8".
I'm not sure if this is the proper way to address this issue, but I've attached a patch that just basically adds the column names explicitly to the SELECT part of the statement.
Comment | File | Size | Author |
---|---|---|---|
#4 | 1053200-pgsql_groupby-3.patch | 1.67 KB | ChrisLaFrancis |
#1 | 1053200-pgsql_groupby-2.patch | 1.73 KB | ChrisLaFrancis |
boost-group_by.patch | 1.73 KB | ChrisLaFrancis | |
Comments
Comment #1
ChrisLaFrancis CreditAttribution: ChrisLaFrancis commentedRe-rolled patch for latest dev and also -p1.
Comment #2
ChrisLaFrancis CreditAttribution: ChrisLaFrancis commentedComment #3
bgm CreditAttribution: bgm commentedThe patch looks OK, but what's the point of the "group by" in the first place? The code seems to be doing that just to get "distinct" results?
Comment #4
ChrisLaFrancis CreditAttribution: ChrisLaFrancis commentedGood point. GROUP BY replaced by DISTINCT.
Comment #5
bgm CreditAttribution: bgm commentedThanks, patch committed to 6.x-1.x.