LakshSingla commented on code in PR #13952:
URL: https://github.com/apache/druid/pull/13952#discussion_r1184941668
##########
processing/src/main/java/org/apache/druid/frame/write/FrameWriters.java:
##########
@@ -53,24 +54,38 @@ private FrameWriters()
* @param signature signature of the frames
* @param sortColumns sort columns for the frames. If nonempty, {@link
FrameSort#sort} is used to sort the
* resulting frames.
+ * @param allowNullColumnTypes to allow null ColumnType in the signature.
This should only be enabled when the user
+ * knows that the column objects exist as native
Java POJOs (LinkedList, Maps etc), which
+ * can be serded using the Druid's nested columns
*/
public static FrameWriterFactory makeFrameWriterFactory(
final FrameType frameType,
final MemoryAllocatorFactory allocatorFactory,
- final RowSignature signature,
- final List<KeyColumn> sortColumns
+ @Nonnull final RowSignature signature,
Review Comment:
Thanks for the pointer! Will fix the clutter.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
