Skip to content

ATLAS-5374: Remove Related Categories / Parent column from glossary AUDIT export (CSV/XLSX) - #725

Open
UmeshPatil-1 wants to merge 1 commit into
apache:masterfrom
UmeshPatil-1:ATLAS-5374
Open

ATLAS-5374: Remove Related Categories / Parent column from glossary AUDIT export (CSV/XLSX)#725
UmeshPatil-1 wants to merge 1 commit into
apache:masterfrom
UmeshPatil-1:ATLAS-5374

Conversation

@UmeshPatil-1

@UmeshPatil-1 UmeshPatil-1 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This PR removes the "Related Categories / Parent" column from glossary AUDIT mode export files (CSV and XLSX).

Changes in this PR

File Change
repository/.../GlossaryExportWriter.java Remove "Related Categories / Parent" from AUDIT_HEADERS; stop writing relatedCategoriesOrParent in toAuditRow()
repository/.../GlossaryExportWriterTest.java Assert CSV output does not contain the removed header; assert XLSX AUDIT export has 10 columns (Qualified Name at index 8, GUID at index 9)

AUDIT export columns after this fix (10 total)

  1. Record Type
  2. Name
  3. Glossary Name
  4. Short Description
  5. Long Description
  6. Status
  7. Classifications
  8. Custom Attributes
  9. Qualified Name
  10. GUID

How was this patch tested?

Unit tests

mvn install -pl intg -DskipTests -Dcheckstyle.skip=true -Drat.skip=true
mvn test -pl repository -Dtest=GlossaryExportWriterTest \
  -Dcheckstyle.skip=true -Drat.skip=true

Results

  • testWriteAuditCsv — PASS; CSV output does not contain "Related Categories / Parent".
  • testWriteAuditXlsx — PASS; XLSX header has 10 columns; column 8 = Qualified Name, column 9 = GUID.

Manual / integration tests (curl)

After mvn install -pl intg,repository,webapp -am and Atlas restart:

UI

No UI code changes in this PR.


@UmeshPatil-1 UmeshPatil-1 changed the title ATLAS-5374: drop Related Categories / Parent column from CSV/XLSX dow… ATLAS-5374: Remove Related Categories / Parent column from glossary AUDIT export (CSV/XLSX) Aug 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant