[
https://issues.apache.org/jira/browse/BEAM-11538?focusedWorklogId=530900&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-530900
]
ASF GitHub Bot logged work on BEAM-11538:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 04/Jan/21 21:14
Start Date: 04/Jan/21 21:14
Worklog Time Spent: 10m
Work Description: Fokko commented on pull request #13635:
URL: https://github.com/apache/beam/pull/13635#issuecomment-754222836
Thanks @kennknowles for the review. I agree that it would be nice to have
more standard Java structures in there by default. I had a simple combiner,
where I changed from a List to a Deque, but it failed to run with a very
hard-to-understand error message and took me quite a while to figure out what
was going on.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 530900)
Time Spent: 1h 20m (was: 1h 10m)
> Add a Deque Encoder
> -------------------
>
> Key: BEAM-11538
> URL: https://issues.apache.org/jira/browse/BEAM-11538
> Project: Beam
> Issue Type: Improvement
> Components: sdk-java-core
> Affects Versions: 2.26.0
> Reporter: Fokko Driesprong
> Assignee: Fokko Driesprong
> Priority: P2
> Time Spent: 1h 20m
> Remaining Estimate: 0h
>
> Next to the List encoder, I think it is nice to have a Deque encode as well.
> The Deque is more performant compared to Lists, and can be used in a
> CombineFn.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
