synapse-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hiranya Jayathilaka (JIRA)" <>
Subject [jira] Reopened: (SYNAPSE-433) Loadbalance Algorithm is hardcoded
Date Tue, 23 Dec 2008 04:29:44 GMT


Hiranya Jayathilaka reopened SYNAPSE-433:

I see two problems in the current load balance algorithm factory implementations.

1. round robin algorithm is still hard coded
2. algorithm factory does not read the algorithm implementation from the 'algorithm' attribute
in the configuration.

Here is the erroneous method from the source.

public static LoadbalanceAlgorithm createLoadbalanceAlgorithm(OMElement loadbalanceElement,
List endpoints) {

        LoadbalanceAlgorithm algorithm = null;

        String algorithmName = "roundRobin";
        OMAttribute algoAttribute = loadbalanceElement.getAttribute(new QName(null, XMLConfigConstants.ALGORITHM_NAME));
        if(algoAttribute != null) {
            algorithmName = algoAttribute.getAttributeValue();

        if(algorithmName.equalsIgnoreCase("roundRobin")) {
                algorithm = new RoundRobin(endpoints);

        return algorithm;

The value of the ALGORITHM_NAME constant is "policy". So the method does not refer the "algorithm"
attribute anywhere in the code.

> Loadbalance Algorithm is hardcoded
> ----------------------------------
>                 Key: SYNAPSE-433
>                 URL:
>             Project: Synapse
>          Issue Type: Bug
>          Components: Endpoints
>            Reporter: Afkham Azeez
>            Assignee: Ruwan Linton
>            Priority: Critical
>             Fix For: NIGHTLY
>         Attachments: lb-algorithm.patch
> There is no way to switch to a different algorithm other than round-robin without a code
change. The load balancing has been hard coded to use round-robin algorithm.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message