ATLAS-5029 : Connected export : When multiple entities are exported ,…

[mandarambawane]

… not all of its connected entities are exported

What changes were proposed in this pull request?

(Please fill in changes proposed in this fix. Create an issue in ASF JIRA before opening a pull request and
set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. ATLAS-XXXX: Fix a typo in YYY))

How was this patch tested?

(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

[Copilot]

Pull Request Overview

This PR addresses an issue with exporting connected entities where not all related entities were properly exported. It adds new tests in ExportServiceTest.java to cover connected export scenarios and introduces logic changes in ExportService.java and RelationshipAttributesExtractor.java to better handle the starting entity in the export process.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
repository/src/test/java/org/apache/atlas/repository/impexp/ExportServiceTest.java	Adds new tests and data provider to verify connected export behavior and proper import processing via ImportService.
repository/src/main/java/org/apache/atlas/repository/impexp/RelationshipAttributesExtractor.java	Updates connected fetch logic to exclude the starting entity from duplicate processing.
repository/src/main/java/org/apache/atlas/repository/impexp/ExportService.java	Introduces a startingEntityGuid to resume processing for the starting entity and adjusts the export traversal loop accordingly.

Comments suppressed due to low confidence (2)

repository/src/main/java/org/apache/atlas/repository/impexp/ExportService.java:292

• Consider adding an inline comment here explaining the purpose of the resumeExportForStartingEntity flag to enhance code clarity during export processing of the starting entity.

if ((context.fetchType == ExportFetchType.CONNECTED ...

repository/src/main/java/org/apache/atlas/repository/impexp/RelationshipAttributesExtractor.java:56

• [nitpick] A short comment explaining the rationale behind resetting the traversal direction for the starting entity could improve clarity for reviewers and maintainers.

if (context.startingEntityGuid.equals(entity.getGuid())) {

[pareshddevalia]

LGTM