[
https://issues.apache.org/jira/browse/TINKERPOP-3082?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17851891#comment-17851891
]
ASF GitHub Bot commented on TINKERPOP-3082:
-------------------------------------------
kenhuuu commented on code in PR #2634:
URL: https://github.com/apache/tinkerpop/pull/2634#discussion_r1625338383
##########
spark-gremlin/src/test/java/org/apache/tinkerpop/gremlin/spark/structure/SparkTest.java:
##########
@@ -51,6 +51,15 @@
* @author Marko A. Rodriguez (http://markorodriguez.com)
*/
public class SparkTest extends AbstractSparkTest {
+ @Test
+ public void testCustomizedSparkAppName() {
Review Comment:
Minor nit: could add a similar test to make sure that the default name is
"Apache TinkerPop's Spark-Gremlin"
> Tinkerpop hardcoded the Spark AppName
> -------------------------------------
>
> Key: TINKERPOP-3082
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3082
> Project: TinkerPop
> Issue Type: Improvement
> Affects Versions: 3.7.2
> Reporter: Redriver
> Priority: Major
> Attachments: Screenshot 2024-06-03 at 16.14.17.png
>
>
> When I run gremlin through SparkGraphComputer, the SparkUI always displays
> the Spark App as "[Apache TinkerPop's
> Spark-Gremlin|http://hdc49-mcc10-01-0510-3405-002-tess0097.stratus.rno.ebay.com:4040/]";
> (see
> https://github.com/apache/tinkerpop/blob/master/spark-gremlin/src/main/java/org/apache/tinkerpop/gremlin/spark/structure/Spark.java#L51)
> which confused users if I run multiple gremlin with SparkGraphComputer.
> It should allow override the Spark App name if user wants to. If user does
> not specify the Spark app name, we can provide the default one.
>
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
