datafusion-comet icon indicating copy to clipboard operation
datafusion-comet copied to clipboard

feat: Add random row generator in data generator

Open advancedxy opened this issue 1 year ago • 1 comments

Which issue does this PR close?

Closes #.

Rationale for this change

Follow up of #426, supports generating random rows for specified struct type.

What changes are included in this PR?

Add utility method: generateRows to generate random rows.

How are these changes tested?

N/A.

advancedxy avatar May 20 '24 06:05 advancedxy

@andygrove would you mind to take a look at this?

advancedxy avatar May 20 '24 06:05 advancedxy

LGTM overall. I think this is a good addition. I left a couple of small style suggestions.

andygrove avatar May 22 '24 17:05 andygrove

Thanks everyone for reviewing.

advancedxy avatar May 24 '24 02:05 advancedxy