[metadata transformation] better error handling in XSLT transformation #253

Open
claudio.atzori wants to merge 3 commits from transformation_wf into beta
Owner

This PR addresses #252 by improving the error handling in the XSLT-based transformation workflow, separating the XSLT creation errors from the XSLT application, and letting the 1st propagate. Furthermore, every error is gathered in the AggregationReport object and reported back to the MSRO.

This PR addresses #252 by improving the error handling in the XSLT-based transformation workflow, separating the XSLT creation errors from the XSLT application, and letting the 1st propagate. Furthermore, every error is gathered in the `AggregationReport` object and reported back to the MSRO.
claudio.atzori added the
enhancement
bug
labels 2 years ago
claudio.atzori self-assigned this 2 years ago
claudio.atzori added 1 commit 2 years ago
claudio.atzori added 1 commit 2 years ago
claudio.atzori requested review from sandro.labruzzo 2 years ago
claudio.atzori requested review from alessia.bardi 2 years ago
Owner

Let's test how it goes, to make sure

  • we do not get any unexpected null pointers for the counters in the report
  • the transformation fails at the first record in case the mapping is not well formed
Let's test how it goes, to make sure - we do not get any unexpected null pointers for the counters in the report - the transformation fails at the first record in case the mapping is not well formed
claudio.atzori added 1 commit 2 years ago

Reviewers

sandro.labruzzo was requested for review 2 years ago
alessia.bardi was requested for review 2 years ago
This pull request has changes conflicting with the target branch.
  • dhp-workflows/dhp-aggregation/src/main/java/eu/dnetlib/dhp/transformation/xslt/XSLTTransformationFunction.java
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b transformation_wf beta
git pull origin transformation_wf

Step 2:

Merge the changes and update on Gitea.
git checkout beta
git merge --no-ff transformation_wf
git push origin beta
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: D-Net/dnet-hadoop#253
Loading…
There is no content yet.