In previous refactor work of visitor, I found that this segment has no
sense in parser, therefore I agree with you that to remove them to simply
our parsing module and integrated tests.
On Thu, Feb 13, 2020 at 2:44 PM Juan Pan wrote:
> Hi everyone,
>
>
> I found the class member
It is good to remove them for unused code.
--
Liang Zhang (John)
Apache ShardingSphere & Dubbo
Juan Pan 于2020年2月14日周五 下午8:55写道:
> Hi
>
>
> I assumed this is a lazy consensus, so PR [1] was raised to fix this
> issue.
>
>
> If you have any objection, please speak up.
>
>
> [1]
Hi
I assumed this is a lazy consensus, so PR [1] was raised to fix this issue.
If you have any objection, please speak up.
[1] https://github.com/apache/incubator-shardingsphere/pull/4290
Juan Pan (Trista)
Senior DBA & PPMC of Apache ShardingSphere(Incubating)