Github user ppadma commented on a diff in the pull request:
https://github.com/apache/drill/pull/1228#discussion_r184483395
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/record/RecordBatchSizer.java
---
@@ -536,6 +556,11 @@ public ColumnSize getColumn(String name) {
*/
private int netRowWidth;
private int netRowWidthCap50;
+
+ /**
+ * actual row size if input is not empty. Otherwise, standard size.
+ */
+ private int rowAllocSize;
--- End diff --
@paul-rogers yes, this PR is trying to do the second thing i.e. make best
guess with no knowledge at all about lengths. Also, right side can produce no
rows for reasons other than empty array i.e. for ex. we might be filtering
everything out after unnest.
Please let me know what you think about the approach.
---
