-
-
Notifications
You must be signed in to change notification settings - Fork 659
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
Fix: IndexOutOfBoundsException thrown in topics view when changing to… #962
Conversation
… cluster with fewer pages [tchiotludo#835]
@tchiotludo If you can review my PR I would appreciate it, I await your comments, greetings! |
@EdwinFajardoBarrera it seems to be a good fix ! |
Sorry I don't get you, how can I help you? |
For example this one, I've the feeling the bug is also here ? |
For sure, take it for granted |
@tchiotludo I'm having issues configuring multiple clusters, actually, this is working
I leave my docker-compose-multiple-cluster.yml file: https://codeshare.io/1YbNvX My application-dev.yml connections are as follows:
Thanks for your help |
can you give the log of the failed connect please ? |
I just got this: I see that the first connect instance that I select makes the other shutdown.
|
You have a mistake corrected here : https://codeshare.io/1YbNvX |
THanks for having a deeper look 👍 |
No problem, let me know if there is another issue with priority (If this is in the backend is better for me) |
backend
|
… cluster with fewer pages.
Closes issue #835
When the cluster is changed now it forces the state to have pageNumber equals to 1 to avoid failure if the new cluster has fewer pages than the old one.