curveadm
curveadm copied to clipboard
[recommend] replace all select * with specific fields.
Using select * might leave a lot of potential problems,. Such as when appending a field in the future, the field will mismatch the code in the old version and result in crushing.
It may be appropriate to replace *
with all fields. cc @caoxianfei1 @Wine93
It may be appropriate to replace
*
with all fields. cc @caoxianfei1 @Wine93
I agree!
I got it, I'll open a pull request soon.