Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix issue 48 #71

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Fix issue 48 #71

wants to merge 5 commits into from

Commits on Apr 16, 2024

  1. Update MultiplyBy2 test

    @LennoxLiu
    LennoxLiu committed Apr 16, 2024
    Configuration menu
    Copy the full SHA
    67166b7 View commit details
    Browse the repository at this point in the history

  2. Check PORT in Makefile run 

    Adding this because there's no error message when failed to start the load-balancer because of port in use

Also add HQ_SUBMIT_DELAY_MS for more stable performance
    @LennoxLiu
    LennoxLiu committed Apr 16, 2024
    Configuration menu
    Copy the full SHA
    eff2e5a View commit details
    Browse the repository at this point in the history

  3. Fix a typo in LoadBalancer.cpp

    @LennoxLiu
    LennoxLiu committed Apr 16, 2024
    Configuration menu
    Copy the full SHA
    a4ba4a0 View commit details
    Browse the repository at this point in the history

  4. Add job.sh for testing in MultiplyBy2

    @LennoxLiu
    LennoxLiu committed Apr 16, 2024
    Configuration menu
    Copy the full SHA
    54b6bab View commit details
    Browse the repository at this point in the history

  5. Apply fix for issue 48 

    Added a timeout when waiting for server to respond. If a timeout happens, rerun job.sh to restart the server.
    @LennoxLiu
    LennoxLiu committed Apr 16, 2024
    Configuration menu
    Copy the full SHA
    1717c6d View commit details
    Browse the repository at this point in the history