Andrew Ash

Results 53 issues of Andrew Ash

## Before this PR Parsing a too-long UUID string fails with an IllegalArgumentException which does not include the invalid value: ``` java.lang.IllegalArgumentException: UUID string too large at java.base/java.util.UUID.fromString1(UUID.java:264) at java.base/java.util.UUID.fromString(UUID.java:258)...

## Before this PR When seeing "Error handling request" in logs, it is difficult to distinguish if the problem is a thrown Error or a thrown ServiceException. ## After this...

## What happened? AssertionsForClassTypes exists as a near-duplicate of the assertj Assertions class. My understanding is that this is due to a deficiency in the Java type system in earlier...

In https://github.com/palantir/streams/pull/36 we marked two methods as deprecated: - MoreStreams#inCompletionOrder (the overload without an `Executor`) - MoreStreams#blockingStreamWithParallelism (the overload without an `Executor`) Uses of those two methods still remain in...

I'm hoping to instrument an implementation of the [Hadoop FileSystem](https://github.com/apache/hadoop/blob/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/FileSystem.java) abstract class, but the current version of Tritium only supports instrumenting interfaces (older versions briefly supports abstract class instrumentation). Please...

## What happened? When creating an sls service or sls asset, the gradle plugins create a `product-dependencies.lock` to show changes and drive some internal tooling that prevents too-rapid pdep upgrades....

Compare these two links: - "Options That Won’t Work " section of https://www.azul.com/jdk-9-pitfalls-for-the-unwary/ - args we set at https://github.com/palantir/sls-packaging/blob/2.4.0/gradle-sls-packaging/src/main/groovy/com/palantir/gradle/dist/service/tasks/LaunchConfigTask.groovy#L34 We might need to remove: - PrintGCDateStamps - UseGCLogFileRotation - GCLogFileSize...

## Before this PR I mistakenly thought that com.palantir.baseline-exact-dependencies being applied (by default) meant that that `check` would run `checkUnusedDependencies`. Actually it does not, and repos wanting this behavior need...

no changelog

## What happened? Entries in `.palantir/revapi.yml` have their versions sorted by string-sort instead of semver-sort. Example: https://github.com/palantir/atlasdb/blob/10fa2981fda1352cd450614338291d741cf73c79/.palantir/revapi.yml#L77-L109 ## What did you want to happen? Sort the versions by semver-sort. Last...