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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ChrisLaFrancis’s picture

Re-rolled patch for latest dev and also -p1.

ChrisLaFrancis’s picture

Status: Active » Needs review
bgm’s picture

The 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?

ChrisLaFrancis’s picture

Good point. GROUP BY replaced by DISTINCT.

bgm’s picture

Status: Needs review » Fixed

Thanks, patch committed to 6.x-1.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.