2

我继承了将生产 MySQL DB 转换为 Postgres 的需要。这主要是使用简单的 SQL 语句来创建表/函数(使用 Navicat 生成半自动转换)来处理的,但现在我在转换有点复杂的视图时遇到了问题。

研究表明,这可能是由于两个 DB 处理子查询(WHERE 语句)的方式不同,也许这只是语法上的差异。这里的业务逻辑是未知的,因为代码库是从另一个开发人员那里继承的。

运行以下命令(使用 Laravel 迁移/PHP 脚本):

SELECT 
parent.is_owner AS is_owner,
parent.brand AS first_name,
parent.id AS id,
(SELECT count(c.id)
 FROM campaigns c
 WHERE((
       (c.user_id = parent.id)
       OR
       (c.user_id = child.id)
       )
       AND
       (c.campaign_status_id = 4)
))
AS current_campaigns,
(SELECT count(c.id)
    FROM campaigns c
    WHERE
        ((
        (c.user_id = parent.id)
        OR (c.user_id = child.id)
        )
        AND (c.campaign_status_id = 5)
))
AS past_campaigns,
(SELECT count(c.id)
    FROM campaigns c
    WHERE
        ((
         (c.user_id = parent.id)
         OR (c.user_id = child.id))
         AND (c.campaign_status_id = 2)
        ))
    AS pending_campaigns,
(SELECT count(c.id)
    FROM    campaigns c
    WHERE ((
            (c.user_id = parent.id)
            OR (c.user_id = child.id)
            )
            AND (c.invoice_status = '1')
        ))
    AS past_invoices
FROM ((users parent LEFT JOIN campaigns mc ON
     ((parent.id = mc.user_id)))
    LEFT JOIN users child ON ((child.parent_owner = parent.id)
    ))
WHERE
(
    (parent.is_owner = 1)
    OR (child.is_retailer = 1)
)
GROUP BY parent.id
ORDER BY parent.brand

...触发错误

SQLSTATE[42803]: Grouping error: 7 ERROR:  subquery uses ungrouped column "child.id" from outer query
  LINE 1: ...c where (((c.user_id = parent.id) or (c.user_id = child.id)) ...

谁能建议如何格式化这个以便 Postgres 运行子查询?

顺便说一句,这里在 Laravel 迁移脚本中使用的 PHP 代码是:

...

DB::unprepared("CREATE VIEW client AS
select parent.is_owner AS is_owner,parent.brand AS first_name,parent.id AS id
   ,(select count(c.id) from campaigns c where (((c.user_id = parent.id) or (c.user_id = child.id)) and (c.campaign_status_id = 4))) AS current_campaigns
   ,(select count(c.id) from campaigns c where (((c.user_id = parent.id) or (c.user_id = child.id)) and (c.campaign_status_id = 5))) AS past_campaigns
   ,(select count(c.id) from campaigns c where (((c.user_id = parent.id) or (c.user_id = child.id)) and (c.campaign_status_id = 2))) AS pending_campaigns
   ,(select count(c.id) from campaigns c where (((c.user_id = parent.id) or (c.user_id = child.id)) and (c.invoice_status = '1'))) AS past_invoices
from ((users parent
left join campaigns mc on((parent.id = mc.user_id)))
left join users child on((child.parent_owner = parent.id)))
where ((parent.is_owner = 1) or (child.is_retailer = 1))
group by parent.id
order by parent.brand;");

更新,固定:

杰出的。来自所有人的非常好的输入。

@patrick 和 @ErwinBrandstetter 的解决方案都有效。我会支持帕特里克在这里,因为我的角色是“按原样”转换系统。未来可能会有重构的空间,但在这个阶段,我觉得弄乱(或改进)别人的管道胶带解决方案是有风险的(即代码库在某些地方似乎过于复杂,没有文档的迹象,我在没有更多关于业务逻辑的背景信息的情况下,我不愿意四处寻找或尝试核心改进)。我怀疑模型的某些部分可能无论如何都需要大修,所以[原文如此]-fix 在这里受到青睐。

我怀疑一些点击抖动可能已经生成了原始查询......试图让原始开发人员从怀疑中受益,并假设存在一些需要快速(即鼠标)周转的业务压力。复杂的 SQL 不是我的强项,但我很高兴我的直觉是正确的,查询一开始就没有必要复杂。也许这个视图是一个计划外的螺栓 - 不是一开始就设计的。明智与否,我可能会尝试使用基于 ORM 的方法来解决问题。

我最后一分钟在这个项目上,运行清理以重新启动(原始开发是“放手”),所以我正在使用一个充满未知功能的大部分未记录的代码库。像伞兵一样奔跑。值得庆幸的是,这个视图问题出现在拼图的最后一块。谢谢 :-)

4

2 回答 2

6

哦,我的,哦,我的。毫无疑问,开发人员的右手无名指有抽搐,因为该语句有不少于 74 个括号。以下是仅使用 8 个括号和 14 行而不是 54 行的方法:

SELECT 
  parent.is_owner AS is_owner,
  parent.brand AS first_name,
  parent.id AS id,
  sum(CASE WHEN c.campaign_status_id = 4 THEN 1 ElSE 0 END) AS current_campaigns,
  sum(CASE WHEN c.campaign_status_id = 5 THEN 1 ElSE 0 END) AS past_campaigns,
  sum(CASE WHEN c.campaign_status_id = 2 THEN 1 ElSE 0 END) AS pending_campaigns,
  sum(CASE WHEN c.invoice_status = '1' THEN 1 ElSE 0 END) AS past_invoices,
FROM users parent
LEFT JOIN users child ON child.parent_owner = parent.id
LEFT JOIN campaigns c ON c.user_id = parent.id OR c.user_id = child.id
WHERE parent.is_owner = 1 OR child.is_retailer = 1
GROUP BY parent.is_owner, parent.brand, parent.id
ORDER BY parent.brand;

没有子选择意味着此代码在启动时会运行得更快。就像 Wolph 在他的评论中提到的那样,选择列表中未包含在聚合函数中的每一列都必须出现在GROUP BY子句中,这是 SQL 标准所指定的,但被 MySQL 忽略了。

CASE通过在列列表中使用构造:条件表达式评估来避免子选择。请注意,子选择中过滤的重复子句现在作为子句执行,主查询中的每列JOIN仅计算一个相关列。从语句中campaigns发出1or并将其包装在函数中是在单个查询中执行多个不同计数的绝妙技巧。0CASEsum()

正如沃尔夫在此答案下方的评论中指出的那样,该条款

sum(CASE WHEN c.campaign_status_id = 4 THEN 1 ElSE 0 END) AS current_campaigns

也可以更简洁地写成

sum((c.campaign_status_id = 4)::integer) AS current_campaigns

CASE考虑到在编写 PostgreSQL 的 C 语言中将布尔值转换为整数不需要任何操作(在 C 中布尔值是 1 或 0),这可能比语句快一些。易读性肯定更少(更不用说使用两倍多的括号了!)。

于 2015-07-18T09:10:53.800 回答
2

问题中缺少解释,但可能的用例是:

计算每个用户“拥有”的广告系列数量。一个用户可以有子用户,子用户的活动应该添加到父用户。

除了@Patrick 在他的演示中整理的令人难以置信的嘈杂语法之外,查询也是模棱两可的(并且可能完全错误):

如果我们可以假设:

  • 参照完整性:子用户仅引用现有的父用户,并通过FOREIGN KEY约束强制执行。

  • 父母和孩子被标记is_owner/is_retailer可靠,那些列只包含值01见下文。

这个查询可以完成这项工作:

SELECT CASE WHEN u.is_retailer = 1 THEN u.parent_owner
            WHEN u.is_owner = 1    THEN u.id END        AS user_id
     , max(u.is_owner)                                  AS is_owner
     , max(u.brand) FILTER (WHERE u.is_owner = 1)       AS first_name
     , count(*) FILTER (WHERE c.campaign_status_id = 4) AS current_campaigns
     , count(*) FILTER (WHERE c.campaign_status_id = 5) AS past_campaigns
     , count(*) FILTER (WHERE c.campaign_status_id = 2) AS pending_campaigns
     , count(*) FILTER (WHERE c.invoice_status = '1')   AS past_invoices
FROM   users          u
LEFT   JOIN campaigns c ON u.id = c.user_id
                       AND (c.campaign_status_id IN (4, 5, 2) OR 
                            c.invoice_status = '1')  -- exclude irrelevant early
WHERE  1 IN (u.is_owner, u.is_retailer)  -- parent & child, may be redundant
GROUP  BY 1
ORDER  BY 2;

应该很快。确保有适合大表的索引。
如果没有其他选项,则此条件是多余的:

   WHERE  1 IN (u.is_owner, u.is_retailer)

我“按原样”使用您的数据模型,但您可能应该只拥有boolean列:

  • is_child:true为孩子,false为父母。
  • is_owner:true对于所有者,false对于零售商。

使用Postgres 9.4 中引入的新聚合子句:FILTER

于 2015-07-18T15:34:28.947 回答