On Tue, 30 Nov 2021 23:03:48 GMT, Paul Sandoz <psan...@openjdk.org> wrote:

>> kabutz has updated the pull request incrementally with one additional commit 
>> since the last revision:
>> 
>>   Made forkOrInvoke() method protected to avoid strange compiler error
>
> src/java.base/share/classes/java/math/BigInteger.java line 1878:
> 
>> 1876: 
>> 1877:         @Serial
>> 1878:         private static final long serialVersionUID = 0L;
> 
> I don't think you need to declare these, the instances are never intended to 
> support serialization e.g. in the stream implementation for `AbstractTask` 
> that extends `CountedCompleter` we state:
> 
>  * <p>Serialization is not supported as there is no intention to serialize
>  * tasks managed by stream ops.

Thanks, I see now that it is common for the ForkJoinTasks to be annotated with 
`@SuppressWarnings("serial")`

Changed

-------------

PR: https://git.openjdk.java.net/jdk/pull/6409

Reply via email to