zeroshade commented on code in PR #36163:
URL: https://github.com/apache/arrow/pull/36163#discussion_r1234553891
##########
go/parquet/pqarrow/file_writer.go:
##########
@@ -181,7 +188,10 @@ func (fw *FileWriter) WriteBuffered(rec arrow.Record)
error {
}
// Write an arrow Record Batch to the file, respecting the MaxRowGroupLength
in the writer
-// properties to determine whether or not a new row group is created while
writing.
+// properties to determine whether a new row group is created or not while
writing.
Review Comment:
Let's update this to say `to determine whether the record is broken up into
more than one row group` and add a statement that it will always create at
least one row group for the record, or something like that.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
