luoc

Results 19 comments of luoc

@pjfanning Thank you for receiving this task. No more operation is actually required. Automatically published after the pull request has merged.

@guihui123456 Thank you for the report. We should have such unit test coverage. It may have been the result of JDBC's unique Calcite custom rules.

@cgivre Does your company have such a demand?

@ssainz Hello, It seems to be resolved. #2202

@ssainz YES. Thanks for the reminder. @vdiravka has already started the process of updating Drill shaded Guava.

@eevanwong I will check the questions in your comments tomorrow.

> Hey, could I get someones input on this? This is quite a small thing, but still something I want to make sure. > > ![image](https://user-images.githubusercontent.com/71536798/112657130-78b69600-8e28-11eb-9e94-62e26d65e371.png) > > This is...

> The most common issues that I suppressed involved null values and potential NPE and override hashcode and equals. The thing with the null values is that, most times in...

> The most common issues that I suppressed involved null values and potential NPE and override hashcode and equals. The thing with the null values is that, most times in...

@eevanwong Thanks. I don't like the false-positive (incorrect). It can take a lot of time. 1. About overriding `hashCode` and `equals`. I holding the review. If possible to count the...