ci(filters): add graphql code to backend trigger (#10113)

datahub-project · Mar 22, 2024 · 8cb65bd · 8cb65bd
1 parent a4418f1
commit 8cb65bd
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/.github/actions/ci-optimization/action.yml b/.github/actions/ci-optimization/action.yml
@@ -70,6 +70,7 @@ runs:
             - "metadata-jobs/**"
             - "metadata-service/**"
             - "metadata-utils/**"
+            - "datahub-graphql-core/**"
             - "smoke-test/**"
             - "docker/**"
           kafka-setup:

diff --git a/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/GmsGraphQLEngine.java b/datahub-graphql-core/src/main/java/com/linkedin/datahub/graphql/GmsGraphQLEngine.java
@@ -1183,11 +1183,12 @@ private void configureMutationResolvers(final RuntimeWiring.Builder builder) {
                     new DeleteGlossaryEntityResolver(this.entityClient, this.entityService))
                 .dataFetcher(
                     "updateName", new UpdateNameResolver(this.entityService, this.entityClient))
-                .dataFetcher("addRelatedTerms", new AddRelatedTermsResolver(this.entityService,
-                    this.entityClient))
                 .dataFetcher(
-                    "removeRelatedTerms", new RemoveRelatedTermsResolver(this.entityService,
-                    this.entityClient))
+                    "addRelatedTerms",
+                    new AddRelatedTermsResolver(this.entityService, this.entityClient))
+                .dataFetcher(
+                    "removeRelatedTerms",
+                    new RemoveRelatedTermsResolver(this.entityService, this.entityClient))
                 .dataFetcher(
                     "createNativeUserResetToken",
                     new CreateNativeUserResetTokenResolver(this.nativeUserService))

diff --git a/...rc/main/java/com/linkedin/datahub/graphql/resolvers/glossary/AddRelatedTermsResolver.java b/...rc/main/java/com/linkedin/datahub/graphql/resolvers/glossary/AddRelatedTermsResolver.java
@@ -47,7 +47,6 @@ public CompletableFuture<Boolean> get(DataFetchingEnvironment environment) throw
           if (GlossaryUtils.canManageChildrenEntities(context, parentUrn, _entityClient)) {
             try {
               final TermRelationshipType relationshipType = input.getRelationshipType();
-              final Urn urn = Urn.createFromString(input.getUrn());
               final List<Urn> termUrns =
                   input.getTermUrns().stream().map(UrnUtils::getUrn).collect(Collectors.toList());
               validateRelatedTermsInput(urn, termUrns);