kennknowles commented on code in PR #31087:
URL: https://github.com/apache/beam/pull/31087#discussion_r1583238947
##########
sdks/java/core/src/main/java/org/apache/beam/sdk/values/PCollectionViews.java:
##########
@@ -960,6 +995,179 @@ public void verifyDeterministic() throws
NonDeterministicException {
}
}
+ /**
+ * Implementation which is able to adapt an iterable materialization to a
{@code List<T>}.
+ *
+ * <p>Unlike ListViewFn2, this implementation is optimized for iteration
rather than indexing.
+ *
+ * <p>For internal use only.
+ */
+ public static class ListViewFn3<T> extends ViewFn<IterableView<T>, List<T>> {
Review Comment:
I don't _love_ that these are just named 1, 2, 3...
--
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]
