search

conveyal / analysis-backend

Server component of Conveyal Analysis
http://conveyal.com/analysis
MIT License
23 stars 12 forks source link

Use new r5 ability to run more than one worker in a local cluster #227

Closed abyrd closed 5 years ago

abyrd commented 5 years ago

This is a useful change for testing that depended on a new config property in R5 (which is now available due to the new R5 release).

Starting more than one worker on the same machine will not improve throughput. Each worker tries to use all available cores on its machine. Running multiple workers is just for testing cluster behavior. All workers are on the same IP address (localhost) but only one of the workers will be able to listen for single point messages since the port is hard-wired.

codecov-io commented 5 years ago

Codecov Report

Merging #227 into dev will increase coverage by 0.1%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##                dev    #227     +/-   ##
==========================================
+ Coverage     23.09%   23.2%   +0.1%     
- Complexity      103     104      +1     
==========================================
  Files            62      62             
  Lines          2416    2422      +6     
  Branches        219     220      +1     
==========================================
+ Hits            558     562      +4     
- Misses         1824    1825      +1     
- Partials         34      35      +1
Impacted Files Coverage Δ Complexity Δ
...rc/main/java/com/conveyal/taui/AnalysisServer.java 45.87% <100%> (ø) 5 <0> (ø) :arrow_down:
.../java/com/conveyal/taui/analysis/LocalCluster.java 76.92% <62.5%> (-3.08%) 3 <1> (+1)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8a29ebc...40d92c6. Read the comment docs.