cedar-policy / cedar-java

Java bindings for the Cedar language
https://www.cedarpolicy.com
Apache License 2.0
42 stars 19 forks source link

Consider removing Guava dependecy #86

Open john-h-kastner-aws opened 8 months ago

john-h-kastner-aws commented 8 months ago

Category

Internal refactors/changes

Describe the feature you'd like to request

We use Guava for a few utility functions, but this pulls in a relativity larger dependency for some functions we could reasonably implement our selves.

When making this change, consider the differences between the builtin unmodifiable sets/maps and Guava's immutable data structures. In particular, the builtin types can be mutated by mutating a reference to the underlying set.

Previously removed in #81, but restored in #83.

Describe alternatives you've considered

.

Additional context

No response

Is this something that you'd be interested in working on?