Lari Hotari

Results 996 comments of Lari Hotari

For software supply chain security reasons, I wouldn't like to depend on projects that aren't well known. In Pulsar, there's already https://github.com/apache/pulsar/blob/master/pulsar-client-dependencies-minimized/pom.xml which handles minimizing `fastutil` classes to the ones...

> Don't depend on the full `fastutil` from `-original` modules in Pulsar. Use https://mvnrepository.com/artifact/com.nukkitx.fastutil/fastutil-long-object-maps instead. This dependency isn't available in MavenCentral: https://central.sonatype.com/search?q=%20com.nukkitx.fastutil . It's in another repository. Maven doesn't have...

This test failure might be related to the changes: ``` Error: Failures: Error: org.apache.pulsar.broker.admin.AdminApiSchemaTest.getTopicIntervalStateIncludeSchemaStoreLedger [INFO] Run 1: PASS Error: Run 2: AdminApiSchemaTest.getTopicIntervalStateIncludeSchemaStoreLedger:292 » NotFound V1 namespace [schematest/test] does not exist...

Regarding CVE-2024-53990, it's also fixed in 2.12.4.

#23472 might be a related issue that shows the root cause. there's a pending PR #23487 to address that deadlock issue in shutdown.

> This PR removes the transitive dependency reactor-netty-http from the Azure Data Explorer module. > The module was bringing in io.projectreactor.netty classes that expose the project to [CVE-2025-22227](https://github.com/advisories/GHSA-4q2v-9p7v-3v22), affecting older...

There's [an emulator for Kusto](https://learn.microsoft.com/en-us/azure/data-explorer/kusto-emulator-install), but unfortunately there's no tests that validate the behavior with the emulator. For TestContainers Java, there's no existing module. For .NET there's a [Kusto module](https://testcontainers.com/modules/kusto/)....

Thanks for the contribution @omrifried. There are checkstyle errors. Configuring code style and checkstyle in IntelliJ will help address those. Please check the guide at https://pulsar.apache.org/contribute/setup-ide/#configure-code-style . You can run...

@aweiri1 Do you use the default configuration of `createTopicToRemoteClusterForReplication=true` in broker.conf? Does the Pulsar Admin connectivity work for the target remote cluster from the source cluster? Is the remote cluster...