Gary Gregory

Results 851 comments of Gary Gregory

I agree BC is key, this would be for 6.0. Nice idea using requireNonNull 👍

> The _assertNotNull_ is verbose and superfluous. Just write `Foo foo = bar.doSomething().getFoo();` and [Helpful NullPointerExceptions](https://openjdk.org/jeps/358) show the null cause. Hello @bjmi [JEP 358](https://openjdk.org/jeps/358) is irrelevant here since it requires...

Just FYI, still an issue with the latest from 2.x: ``` git log -1 commit b55c4d3fa2489cf91d6569aac57b05ae2c9d4b06 (HEAD -> 2.x, origin/2.x) ```

I am using Java 17 on Windows 10 with Maven: ``` ver && java -version && mvn -version Microsoft Windows [Version 10.0.19045.4170] openjdk version "17.0.8" 2023-07-18 OpenJDK Runtime Environment Temurin-17.0.8+7...

Both `ResourceAttributes.SERVICE_NAME` and `ResourceAttributes.SERVICE_VERSION` are deprecated, so what's supposed to go in these `put` calls?

Hello @laurit My dependency management section uses: ``` io.opentelemetry opentelemetry-bom 1.37.0 pom import ``` Version "1.37.0" looks like the latest: https://search.maven.org/artifact/io.opentelemetry/opentelemetry-bom My dependency section uses: ``` io.opentelemetry.instrumentation opentelemetry-log4j-appender-2.17 2.3.0-alpha io.opentelemetry...

Hello @laurit 1) Like the title of the ticket says "Exporters example does not compile", so that needs fixing. 2) The offered solution uses deprecate code, so what's the supported...

-1: This looks too hacky and brittle to me as we do not have ways to prevent future changes from depending on java.desktop or other modules which would make this...

Closing: For 4.0, we can remove the dependency on `java.desktop`, probably.

@ppkarwasz Whatever I had noticed before was through a scan from at least one vendor we use at my day job and there is no way to test snapshot or...