[ 
https://issues.apache.org/jira/browse/BEAM-4088?focusedWorklogId=116873&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-116873
 ]

ASF GitHub Bot logged work on BEAM-4088:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 28/Jun/18 16:25
            Start Date: 28/Jun/18 16:25
    Worklog Time Spent: 10m 
      Work Description: kennknowles commented on issue #5280: [BEAM-4088] 
Instrument ExecutorServiceParallelExecutorTest
URL: https://github.com/apache/beam/pull/5280#issuecomment-401093338
 
 
   We have turned on autoformatting of the codebase, which causes small 
conflicts across the board. You can probably safely rebase and just keep your 
changes. Like this:
   
   ```
   $ git rebase
   ... see some conflicts
   $ git diff
   ... confirmed that the conflicts are just autoformatting
   ... so we can just keep our changes are do our own autoformat
   $ git checkout --theirs --
   $ git add -u
   $ git rebase --continue
   $ ./gradlew spotlessJavaApply
   ```
   
   Please ping me if you run into any difficulty. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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: 116873)
    Time Spent: 2h 50m  (was: 2h 40m)

> ExecutorServiceParallelExecutorTest#ensureMetricsThreadDoesntLeak in PR #4965 
> does not pass in gradle
> -----------------------------------------------------------------------------------------------------
>
>                 Key: BEAM-4088
>                 URL: https://issues.apache.org/jira/browse/BEAM-4088
>             Project: Beam
>          Issue Type: Sub-task
>          Components: testing
>            Reporter: Etienne Chauchot
>            Assignee: Etienne Chauchot
>            Priority: Major
>             Fix For: Not applicable
>
>          Time Spent: 2h 50m
>  Remaining Estimate: 0h
>
> This is a new test being added to ensure threads don't leak. The failure 
> seems to indicate that threads do leak.
> This test fails using gradle but previously passed using maven
> PR: https://github.com/apache/beam/pull/4965
> Presubmit Failure: 
>  * https://builds.apache.org/job/beam_PreCommit_Java_GradleBuild/4059/
>  * 
> https://scans.gradle.com/s/grha56432j3t2/tests/jqhvlvf72f7pg-ipde5etqqejoa?openStackTraces=WzBd



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to