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.

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

HTTP API very slow #213

Closed
mcweba opened this issue Aug 22, 2024 · 3 comments
Closed

HTTP API very slow #213

mcweba opened this issue Aug 22, 2024 · 3 comments
Assignees

Comments

@mcweba
Copy link
Collaborator

mcweba commented Aug 22, 2024

The /queuing/monitor and /queuing/statistics HTTP API endpoints in newer vertx-redisques versions are really slow. The other endpoints like /queuing/queues or /queuing/locks are fast in the newer versions too. Check the two compared versions below:

3.0.31

curl -o /dev/null  -s -w "Response Time: %{time_total}s\n"  http://localhost:7018/queuing/monitor
Response Time: 0.006s

4.1.2

curl -o /dev/null  -s -w "Response Time: %{time_total}s\n"  http://localhost:7018/queuing/monitor
Response Time: 0.780s

This is around 130 times slower in the new version. One thing related could be that newer versions have more output. See:

{
    "queues": [
        {
            "name": "nemo-to-dormakaba-v2",
            "size": 151,
            "lastDequeueAttempt": "",
            "lastDequeueSuccess": "",
            "nextDequeueDueTimestamp": ""
        },
        {
            "name": "edds-delivery",
            "size": 50,
            "lastDequeueAttempt": "",
            "lastDequeueSuccess": "",
            "nextDequeueDueTimestamp": ""
        },
        {
            "name": "edds-deliveryEvent-kafka-fwd",
            "size": 17,
            "lastDequeueAttempt": "",
            "lastDequeueSuccess": "",
            "nextDequeueDueTimestamp": ""
        }
    ]
}

The output of the older versions is:

{
    "queues": [
        {
            "name": "nemo-to-dormakaba-v2",
            "size": 3010
        },
        {
            "name": "edds-delivery",
            "size": 111
        },
        {
            "name": "edds-sisy-signatureEvent-kafka-fwd",
            "size": 59
        }
    ]
}
@mcweba
Copy link
Collaborator Author

mcweba commented Aug 26, 2024

QueueStatistics can be disabled by configuration. In this case I would also expect the API to not collect/process this data. Therefore the response times should be as fast as before

mcweba added a commit that referenced this issue Oct 7, 2024
@mcweba
Copy link
Collaborator Author

mcweba commented Oct 7, 2024

Fixed in v4.1.3

@mcweba mcweba closed this as completed Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants