Skip to content

Remove com.google.gson.graph.#2990

Open
eamonnmcmanus wants to merge 2 commits intomainfrom
graph
Open

Remove com.google.gson.graph.#2990
eamonnmcmanus wants to merge 2 commits intomainfrom
graph

Conversation

@eamonnmcmanus
Copy link
Member

@eamonnmcmanus eamonnmcmanus commented Mar 5, 2026

This "extras" package provides a way to represent an arbitrary Java object graph in JSON. However, it is not part of any supported artifact. It is available for use internally at Google, but nothing uses it.

I have been experimenting with possibly changing
ReflectiveTypeAdapterFactory so that it can call a constructor whose parameter names are known, rather than using Unsafe.allocateInstance. However, GraphAdapterBuilderTest fails during deserialization, complaining about recursive construction. Rather than trying to make the test work, I am inclined to delete this unused functionality.

Also remove the extras project from the API compatibility check. We don't release any artifacts from this project, so API compatibility isn't really a concern there.

This "extras" package provides a way to represent an arbitrary Java
object graph in JSON. However, it is not part of any supported artifact.
It is available for use internally at Google, but nothing uses it.

I have been experimenting with possibly changing
`ReflectiveTypeAdapterFactory` so that it can call a constructor whose
parameter names are known, rather than using `Unsafe.allocateInstance`.
However, `GraphAdapterBuilderTest` fails during deserialization,
complaining about recursive construction. Rather than trying to make
the test work, I am inclined to delete this unused functionality.
@eamonnmcmanus eamonnmcmanus requested a review from cpovirk March 5, 2026 20:19
We don't release any artifacts for this project.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants