daison12006013 / laravel-horizon-cluster

A laravel horizon with redis cluster support
MIT License
9 stars 4 forks source link

Batches and Metrics not working? #1

Closed aromka closed 8 months ago

aromka commented 9 months ago

Just trying this out and seems to work great with AWS serverless redis, which uses clusters. Great job on that!

Noticed that Metrics and Batches don't seem to work when you dig deeper into the stats - just keeps loading. Is there anything that need to be adjusted for that?

daison12006013 commented 9 months ago

Here's our metrics in test environment image image image

Make sure to always follow this instruction https://laravel.com/docs/10.x/horizon#metrics Regarding batches, I cannot confirm on this yet, we're not usually using the Batches logic from laravel, although it should not be affected with this, I checked all the pipeline code and it's already pointing to the blocking i/o logic.

aromka commented 8 months ago

Nevermind, this was our internal route setting that was causing the issue, all good now. Thanks for doing great job on this!

daison12006013 commented 8 months ago

@aromka appreciated your testing, hopefully this package can reach out more devs that uses cluster based redis.