Ladislav Thon

Results 347 comments of Ladislav Thon

This is ultimately the same thing as #160. Closing so that the discussion doesn't bifurcate.

Reopening after scope clarification: #160 is about a simple fix of backup/restore of domain.xml config, while this is about enabling backup/restore of host.xml config (which would require more work both...

Hmm, now that's an interesting idea. We could change all command implementations, and make sure that we don't miss anything in the future. That's not gonna work. Or there could...

OK, so we already have a similar convention: builders of the `Add*` commands often have a `replaceExisting()` method. I'd suggest that builders of the `Remove*` commands would have a method...

I actually like `ignoreMissing()` the most, because it's symmetric with `replaceExisting()`.

I'd prefer adding builders for options like this. The existing constructors would have to be preserved, of course.

Then we are in agreement and the only remaining thing is to find someone who will do it :-)

Just for the record, I didn't forget about this. This is basically about canonicalizing the key to the connection pool map. I'd like to improve the code related to connection...

Do you maybe use `Command.create(String)` to create an instance of command that is sent to the Redis cluster? I accidentally found out that we were doing that in Quarkus and...