Duansg commented on issue #3496:
URL: https://github.com/apache/hertzbeat/issues/3496#issuecomment-2998479534

   > In addition, I also support [@Duansg](https://github.com/Duansg)'s 
suggestion to split it into two .g4 files; I recommend we can add this as a 
future development goal. Then, regarding the complex syntax 
[@Duansg](https://github.com/Duansg) mentioned earlier, such as arithmetic 
expressions, which we indeed do not currently support—should its priority be 
set higher than splitting the .g4 files, but lower than the solution I proposed 
above?那么,关于前面提到的复杂语法,比如算术表达式,我们目前确实不支持--它的优先级是否应高于拆分 .g4 文件,但低于我上面提出的解决方案?
   
   I think splitting .g4 files should be done before supporting complex 
expressions, it will be good for extension and maintenance, at this stage we 
only deal with the current problem. But just make sure the syntax parsing is 
independent at this stage.
   
   There's been a lot of discussion about `value==null`, but I don't think 
that's the focus of this installment. There are a lot of ways to handle it, and 
even with the new preview, it can still be filtered from the front and back 
end, so we just need to focus on parsing it correctly and accurately.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@hertzbeat.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@hertzbeat.apache.org
For additional commands, e-mail: notifications-h...@hertzbeat.apache.org

Reply via email to