search

deephaven / deephaven-core

Deephaven Community Core
Other
257 stars 80 forks source link

Consider use of Flight POJOs instead of protobufs #6295

Open devinrsmith opened 3 weeks ago

devinrsmith commented 3 weeks ago

While implementing #5339, I wished I could have used the Flight POJOs (org.apache.arrow.flight) everywhere instead of the Flight protobufs (org.apache.arrow.flight.impl); while it would be a breaking change, it would be relatively straightforward to update all of our code save for the grpc sending layer, to work directly with Flight POJOs.

For example, io.deephaven.server.session.ActionResolver was implemented with org.apache.arrow.flight.Action, org.apache.arrow.flight.ActionType, and org.apache.arrow.flight.Result as opposed to org.apache.arrow.flight.impl.Flight.Action, org.apache.arrow.flight.impl.Flight.ActionType, and org.apache.arrow.flight.impl.Flight.Result.