[Question] sql 最后如果有 注释 -- 那么设置的查询总条数限制就失效了 #1263

Activity

1. Issues-translate-bot commented on Apr 9, 2026

   

   Issues-translate-bot

   on Apr 9, 2026

   More actions

   Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

   ---

   Title: [Question] If there is a comment at the end of the sql -- then the limit on the total number of queries set will be invalid.

   Describe the problem
   If there is a comment at the end of sql -- then the limit on the total number of queries set will be invalid.

   Recurrence
   Steps to reproduce the behavior:

   1. Set the total number of queries to 1000
   2. Find a table A with more than 1000 entries and execute the query select * form A --
   3. All data will be queried. If export permission is given, all data should be exported.

   Desired results
   Even if -- is added to the sql at the end, do not comment out the limit for platform splicing.
   In fact, executing select * form A -- where has the same effect.
2. dosubot commented on Apr 9, 2026

   

   dosubotBot

   on Apr 9, 2026 – with DosuBot

   Last edited by dosubot

   More actions

   Hi @xiaoxiaoadmin! I'm Dosu and I’m helping the Yearning team.

   这个问题确认存在。根源在于 Yearning 在将 SQL 发送给 RPC 引擎执行前，没有对 SQL 中的注释进行清理，导致 RPC 引擎拼接的 LIMIT 子句被 -- 注释掉。

   执行流程大致是：

   1. 用户提交 SELECT * FROM A --
   2. 主应用将 SQL 和配置的 limit 值通过 RPC 发送给引擎服务 [1]
   3. 引擎在 SQL 末尾拼接 LIMIT 1000，变成 SELECT * FROM A -- LIMIT 1000
   4. -- 之后的内容被数据库当作注释，LIMIT 失效

   值得注意的是，Yearning 代码中其实已经有注释剥离的正则逻辑（支持 -- 单行注释和 /* */ 多行注释），但目前只在 GetFingerprint() 中用于 SQL 指纹提取和 AI 建议功能 [2]，并没有应用在查询执行路径上。

   可能的修复方向：

   • 在主应用侧：将 SQL 发送给 RPC 引擎前，复用 GetFingerprint() 中的注释剥离逻辑清理 SQL
   • 在 RPC 引擎侧：在拼接 LIMIT 前先移除尾部注释
   • 或者更稳妥地：解析 SQL 结构后将 LIMIT 插入到注释之前

   /* */ 块注释理论上也有类似的绕过风险。

   To reply, just mention @dosu.

   ---

   Docs are dead. Just use Dosu.