Skip to content
This repository has been archived by the owner on Feb 8, 2019. It is now read-only.

fix GEARPUMP-155, integration test failure #36

Closed
wants to merge 1 commit into from

Conversation

manuzhang
Copy link
Contributor

Changes include:

  1. fix integration test
  2. DistShellAppMaster and DistServiceAppMaster send ActivateAppMaster(appId) message to master when all the executor systems have been launched
  3. fix warning

@codecov-io
Copy link

Current coverage is 65.56%

Merging #36 into master will increase coverage by 1.21%

  1. 7 files (not in diff) in ...apache/gearpump/util were modified. more
    • Misses -6
    • Hits +6
  2. 2 files (not in diff) in ...rpump/cluster/client were modified. more
    • Misses -43
    • Hits +43
  3. 5 files (not in diff) in ...che/gearpump/cluster were modified. more
    • Misses -20
    • Hits +20
  4. File ...ntStoreFactory.scala (not in diff) was modified. more
    • Misses -2
    • Partials 0
    • Hits +2
  5. File ...cutor/Executor.scala (not in diff) was modified. more
    • Misses -1
    • Partials 0
    • Hits +1
@@             master        #36   diff @@
==========================================
  Files           178        178          
  Lines          5922       5924     +2   
  Methods        5644       5645     +1   
  Messages          0          0          
  Branches        278        279     +1   
==========================================
+ Hits           3811       3884    +73   
+ Misses         2111       2040    -71   
  Partials          0          0          

Sunburst

Powered by Codecov. Last updated by ae32049...0322be3

@huafengw
Copy link
Contributor

huafengw commented Jun 7, 2016

+1

@asfgit asfgit closed this in 97bf112 Jun 7, 2016
@manuzhang manuzhang deleted the GERRPUMP-155 branch June 13, 2016 07:23
manuzhang pushed a commit to manuzhang/incubator-gearpump that referenced this pull request Jan 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants