[jira] [Commented] (COMPRESS-430) Synchronize iteration on synchronized list in ParallelScatterZipCreator

[ASF GitHub Bot (JIRA)]

    [ 
https://issues.apache.org/jira/browse/COMPRESS-430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16265655#comment-16265655
 ] 

ASF GitHub Bot commented on COMPRESS-430:
-----------------------------------------

Github user coveralls commented on the issue:

    https://github.com/apache/commons-compress/pull/57
  
    
    [![Coverage 
Status](https://coveralls.io/builds/14361284/badge)](https://coveralls.io/builds/14361284)
    
    Coverage increased (+0.05%) to 85.932% when pulling 
**c0499042c2778161d6cecd54357adb6ea9595132 on 
kinow:synchronize-iteration-of-synchronized-list-1** into 
**10ff1c341d37da1cdb2a9ecb280a7e73670c00f5 on apache:master**.



> Synchronize iteration on synchronized list in ParallelScatterZipCreator
> -----------------------------------------------------------------------
>
>                 Key: COMPRESS-430
>                 URL: https://issues.apache.org/jira/browse/COMPRESS-430
>             Project: Commons Compress
>          Issue Type: Bug
>          Components: Archivers
>    Affects Versions: 1.15
>            Reporter: Bruno P. Kinoshita
>            Priority: Minor
>              Labels: concurrency
>             Fix For: 1.16
>
>
> Following what is defined in the [Collections 
> javadoc|https://docs.oracle.com/javase/9/docs/api/java/util/Collections.html#synchronizedList(java.util.List)],
>  it might be a good idea to synchronize when iterating the stream list, that 
> is created with Collections.synchronizedList.
> This work originated after the pull request submitted to VALIDATOR-434, and 
> searching other components in commons using similar method to create lists 
> but not synchronizing the iteration.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)