Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating unit tests based on new typeMetadata format #2286

Open
wants to merge 19 commits into
base: integration
Choose a base branch
from

Conversation

mineralntl
Copy link
Collaborator

This branch will fail until the corresponding typeMetadata serialization changes from the metdata-utils repo are merged and tagged - the metadata-utils version in this branch's pom (or whatever is merged into integration) will also need to be updated accordingly.

avgAGB
avgAGB previously approved these changes Feb 27, 2024
ivakegg
ivakegg previously approved these changes Feb 27, 2024
@mineralntl
Copy link
Collaborator Author

@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch from fb22cdf to b0da3c6 Compare February 29, 2024 14:44
…tionalSecurityAgency/datawave into task/typeMetadataSerializationSupport
hlgp
hlgp previously approved these changes Mar 4, 2024
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch from 88cd445 to 5137d97 Compare March 7, 2024 20:08
…tionalSecurityAgency/datawave into task/typeMetadataSerializationSupport
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch from 4971a61 to 9991da1 Compare March 18, 2024 13:16
@mineralntl mineralntl marked this pull request as ready for review March 21, 2024 01:27
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch 2 times, most recently from 5edf100 to d58c82b Compare March 25, 2024 02:59
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch from d58c82b to 13c8fac Compare April 3, 2024 17:39
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch 2 times, most recently from 1173287 to 876113f Compare April 17, 2024 14:46
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch 3 times, most recently from 643fc9b to 0d3eeef Compare April 26, 2024 16:31
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch from 0d3eeef to ca455dd Compare May 9, 2024 13:38
@mineralntl mineralntl force-pushed the task/typeMetadataSerializationSupport branch from ca455dd to 695bf60 Compare May 17, 2024 16:38
@mineralntl mineralntl dismissed stale reviews from hlgp, avgAGB, and ivakegg via f99e2a3 May 21, 2024 16:11
@mineralntl
Copy link
Collaborator Author

This PR is technically optional due to the ability to consume and parse the previously used TypeMetadata string format. Merge (or not) whenever we feel like it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants