Skip to content
This repository has been archived by the owner on Jun 7, 2024. It is now read-only.

Support making DataLake annotations as mandatory upon creation and update of event type #1587

Merged
merged 4 commits into from
Feb 5, 2024

Conversation

1u0
Copy link
Member

@1u0 1u0 commented Jan 25, 2024

One-line summary

Zalando ticket : ARUHA-XXX (only if appropriate)

Description

A few sentences describing the overall goals of the pull request's
commits.

Review

  • Tests
  • Documentation

Deployment Notes

These should highlight any db migrations, feature toggles, etc.

@1u0 1u0 force-pushed the datalake-annotations branch 4 times, most recently from 0c87547 to 2583196 Compare January 26, 2024 12:51
@1u0 1u0 force-pushed the datalake-annotations branch 3 times, most recently from 2a9c54d to b4bcb7f Compare January 31, 2024 09:54
@1u0 1u0 changed the title Refactor DataLake annotations validation code Support making DataLake annotations as mandatory upon creation and update of event type Feb 2, 2024
@1u0 1u0 marked this pull request as ready for review February 2, 2024 10:41
@1u0
Copy link
Member Author

1u0 commented Feb 5, 2024

👍

" the error message should include the retention reason annotation name",
e.getMessage().contains(EventTypeAnnotationsValidator.RETENTION_REASON_ANNOTATION));
Assert.assertTrue(
"When the retention period is specified but the retention reason is not," +
Copy link
Member

@adyach adyach Feb 5, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"When the retention period is specified but the retention reason is not," +
"When the retention reason is specified but the retention period is not," +

@adyach
Copy link
Member

adyach commented Feb 5, 2024

👍

@1u0 1u0 merged commit 1deaf40 into master Feb 5, 2024
4 checks passed
@1u0 1u0 deleted the datalake-annotations branch February 5, 2024 13:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants