Lari Hotari

/pulsarbot rerun-failure-checks

> Could you please approve the PR to be tested? Is there any concerns? cc @lhotari @nodece @berg223 Not really any concerns, but I just haven't had a chance to...

@berg223 Regarding some other ExtensibleLoadManagerImpl related tests (in ExtensibleLoadManagerImplTest and in ServiceUnitStateChannelTest), one detail is about inconsistencies in how the test code changes the primary/leader by closing LeaderElectionImpl. LeaderElectionImpl should...

> `protobuf-java` contains braking changes, and since protobuf v26, `protobuf-java` v4.x is needed. Can this dependency get upgraded? The protobuf-java version is decoupled from the Pulsar Java client with #23632...

> > `protobuf-java` contains braking changes, and since protobuf v26, `protobuf-java` v4.x is needed. Can this dependency get upgraded? > > The protobuf-java version is decoupled from the Pulsar Java...

> The repo I mention here is the enigo repo, and I think double input is not a good thing. thanks. Found the issue enigo-rs/enigo#464. Looking forward to progress on...

> @lhotari if you get a chance would you mind trying that PR? I don't have a Linux machine at the moment so it's gonna take me a bit to...

Yes, this would be a useful feature for many usecase. This seems to be the only feature that I'm missing when I'm using DeskPad.

btw. It might be the case that the previous code is fine. The problem with the old code is that it is cryptic. In the original code, `signSafeMod` function's name...

> If this is really a bug then the PR needs a test It's not a bug, just bad method naming in the original code. I'll close this.