curveadm icon indicating copy to clipboard operation
curveadm copied to clipboard

[recommend] replace all select * with specific fields.

Open WhereAreBugs opened this issue 1 year ago • 3 comments

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.

WhereAreBugs avatar Dec 15 '23 16:12 WhereAreBugs

It may be appropriate to replace * with all fields. cc @caoxianfei1 @Wine93

montaguelhz avatar Dec 18 '23 04:12 montaguelhz

It may be appropriate to replace * with all fields. cc @caoxianfei1 @Wine93

I agree!

Wine93 avatar Dec 26 '23 06:12 Wine93

I got it, I'll open a pull request soon.

WhereAreBugs avatar Dec 29 '23 03:12 WhereAreBugs