Github user cloud-fan commented on the issue:
    >  This changes equality for the scan and streaming relation, though.
    If we think this is the right equality for `DataSourceV2Relation`, it 
should also be the right equality for scan and streaming relation. They should 
be consistent.
    I've rolled back unnecessary style-only changes, but leave the one that 
clean up unused imports. I think this should be encouraged, it's very hard to 
find out unused imports during code review, people should find and fix them 
while touching files in the PR, some IDE can even do it automatically when you 
save the file.


To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to