mergestat-lite icon indicating copy to clipboard operation
mergestat-lite copied to clipboard

handle null values in `summarize` queries better

Open patrickdevivo opened this issue 2 years ago • 0 comments

https://github.com/mergestat/mergestat/blob/main/cmd/summarize/commits/commits.go#L63-L70

It's possible for author_name / author_email to be null, in which case we have a Scan error - should use sql.NullString in these cases

patrickdevivo avatar Mar 24 '22 14:03 patrickdevivo