Skip to content

refactor: optimize flinksql grammar#480

Open
liuxy0551 wants to merge 1 commit into
DTStack:nextfrom
liuxy0551:feat_optimize_flinksql
Open

refactor: optimize flinksql grammar#480
liuxy0551 wants to merge 1 commit into
DTStack:nextfrom
liuxy0551:feat_optimize_flinksql

Conversation

@liuxy0551

Copy link
Copy Markdown
Collaborator
指标 优化前 优化后 变化
Grammar 行数 1694 1318 -376 行(-22.2%)
删除规则数 14 个
生成 Parser 净减 1129 行
测试 392 通过 402 通过 新增 10 个用例,0 失败
  • 内联单次引用规则,将只用 1 次的规则直接内联到调用处,减少 ATN 状态数:
    emptyStatement、metadataKey、computedColumnExpression、dateTimeExpression、withItemName、notForced、groupingSets、groupingSetsNotationName、partitionDefinition、transformList、insertPartitionDefinition
  • 内联少次引用规则(2 次,体短):jarFileName、timeAttrColumn
  • 删除未使用规则:correlationName
image image

@liuxy0551 liuxy0551 requested a review from Cythia828 June 23, 2026 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant