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

REPO-5191 Bug: T-Engine should provide mapping rather than the repo. #316

Merged
merged 5 commits into from
Jan 6, 2021

Conversation

alandavis
Copy link
Contributor

@alandavis alandavis commented Jan 4, 2021

Bug found while reviewing documents on how to create a custom metadata extractor. The original refactor had left the repo doing the mapping. It should have been passing the fully qualified repo properties to the T-Engine to do the mapping.

Linked to:

Bug found while reviewing documents on how to create a custom metadata extractor. The original refactor had left the repo doing the mapping. It should have been passing the fully qualified repo properties to the T-Engine to do the mapping.
Copy link
Contributor

@eknidev eknidev left a comment

Choose a reason for hiding this comment

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

Looks good to merge once the test tika engine config is updated.

alandavis added a commit to Alfresco/acs-packaging that referenced this pull request Jan 6, 2021
@alandavis alandavis merged commit 2fd11d5 into master Jan 6, 2021
@alandavis alandavis deleted the REPO-5191_metadata_embed_t_engine_to_do_mapping branch January 6, 2021 22:25
alandavis added a commit to Alfresco/alfresco-community-repo that referenced this pull request Jan 7, 2021
…227)

Bug found while reviewing documents on how to create a custom metadata extractor. The original refactor had left the repo doing the mapping. It should have been passing the fully qualified repo properties to the T-Engine to do the mapping.

Linked to:
    Alfresco/acs-packaging#1826
    Alfresco/alfresco-transform-core#316
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.

2 participants