一个 SQL 大家帮忙理解一下

[huzhengnan]

优化老项目，访问人数不多，数据库 CPU100%，内存 100% 发现 filter 中有这么个 SQL，也不止这么个 SQL，但是这个是我看了比较久的一个

SELECT a.*
FROM user a
WHERE a.seller_id = 600000
	AND a.user_type = (
		SELECT MAX(b.user_type)
		FROM user b
		WHERE a.seller_id = b.seller_id
		GROUP BY seller_id
	)
GROUP BY seller_id

问两个问题

1. 可以改为什么 SQL
2. 这么写的思路是什么

[oaix]

SELECT a.*
FROM user a
WHERE a.seller_id = 600000
ORDER BY user_type DESC LIMIT 1

[shanYueFengCheng]

复杂 SQL 分为多次简单查询----《深入理解 Mysql 》

拆分成两次查询。
目标表 user 取个用户类型的最大值
SELECT MAX(user_type) FROM user

获取用户类型的最大值，userType 和销售者 id sellerId
SELECT a.* FROM user a WHERE a.seller_id = ? AND a.user_type = ?
GROUP BY seller_id

[taogen]

@oaix 楼上正解。查询某个 seller_id 中 user_type 最大的一行。那个子查询没必要。

[shanYueFengCheng]

不确定原表设计，不能确定只有一条

[taogen]

@shanYueFengCheng 老哥说得对，在下疏忽了。一楼的回答不一定对，要看业务是不是只需要一条数据。

[oaix]

@taogen 看最后面有个 group by

[jaskle]

无索引的 group by 是锁表利器

[taogen]

@oaix 最后的 group by 没看懂，感觉没什么意义，父查询并不需要使用分组对应的函数如 max，min。
你的 limit 1 感觉不行，如果有多行数据的 user_type = max(user_type) 呢？原结果是多行，你的始终是一行

[txwd]

据我所知，这个 SQL 执行不了，最后来这个 GROUP BY。代码没贴全？

[akira]

1l 应该就对了. 除非 a 表里面有 usertype 是大于 b 表里面的。

原 sql 应该也没有太大问题，怀疑 seller_id 没做索引

[oaix]

@taogen #8

mysql-sql> select * from tbl_test;
+----+------+------+
| id | type | name |
+----+------+------+
| 1 | 1 | a |
| 2 | 1 | b |
| 3 | 2 | c |
| 4 | 2 | d |
+----+------+------+
4 rows in set (0.00 sec)
mysql-sql> select * from tbl_test where type=2;
+----+------+------+
| id | type | name |
+----+------+------+
| 3 | 2 | c |
| 4 | 2 | d |
+----+------+------+
2 rows in set (0.00 sec)
mysql-sql> select * from tbl_test where type=2 group by type;
+----+------+------+
| id | type | name |
+----+------+------+
| 3 | 2 | c |
+----+------+------+
1 row in set (0.00 sec)

[taogen]

@oaix group by 还有这个效果。多谢赐教！

[zjp]

@oaix 有 ONLY_FULL_GROUP_BY 的问题
+----+------+------+
| id | type | name |
+----+------+------+
| 3 | 2 | c |
| 4 | 2 | d |
+----+------+------+
都满足条件，但是 group by 丢弃了一行

[LeeSeoung]

先查 sql 执行时间，找出慢的再优化，不要凭人为感觉哪个 sql 有问题，找到执行慢的 sql 直接看执行计划来进行优化。

[addsionhuey]

select *
from (
SELECT a.*,row_number() over(partition by seller_id order by user_type desc) as ROW_ORDER
FROM user a
WHERE a.seller_id = 600000
)
where ROW_ORDER = 1