I agree with removing this. I've seen users complain about this method not
working properly, but I find it very unexpected that they are using it in
the first place. At the very least it should be made package-private as it
seems to me to be an internal implementation, rather than something users
should have access to.
On Mon, Jun 24, 2024 at 4:12 PM Valentin Kagamlyk <
valentin.kagam...@gmail.com> wrote:
> Hi all,
>
> Currently, it is possible to send a Traversal from the Java GLV using
> Client. For example,
>
> final Graph graph = TinkerGraph.open();
> final GraphTraversalSource g = graph.traversal();
> final Client aliased = client.alias("gmodern");
> final ResultSet resultSetUnrolled = aliased.submit(g.V());
>
> The problem is that this is equivalent to submitting a bytecode which will
> be removed in TP4.
>
> I'm thinking of removing this confusing function. The Client is designed to
> send string Gremlin queries, but for Traversals, DriverRemoteConnection
> should be used instead.
>
> If you want to keep `Client.submit(Traversal)`, then do not hesitate to say
> in the next 72 hours.
>
> Regards,
> Valentyn
>
