xiazcy commented on code in PR #3233:
URL: https://github.com/apache/tinkerpop/pull/3233#discussion_r2421079084
##########
gremlin-test/src/main/java/org/apache/tinkerpop/gremlin/process/traversal/step/sideEffect/AggregateTest.java:
##########
@@ -58,10 +58,6 @@
@RunWith(GremlinProcessRunner.class)
public abstract class AggregateTest extends AbstractGremlinProcessTest {
- ////// global
-
- public abstract Traversal<Vertex, List<String>>
get_g_V_name_aggregateXglobal_xX_capXxX();
Review Comment:
I didn't think there is the need. These process tests duplicated the feature
tests in Java (I think the feature tests were based off of this), which are now
obsolete, especially given we've added remote execution of feature tests for
Java. There is plan to deprecate and remove it completely at some point, and it
hasn't been updated for the past few versions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
