Skip to content

SDE if include/import schemaLocation not found.#744

Merged
mbeckerle merged 1 commit into
apache:mainfrom
mbeckerle:daf-2642-logging
Feb 2, 2022
Merged

SDE if include/import schemaLocation not found.#744
mbeckerle merged 1 commit into
apache:mainfrom
mbeckerle:daf-2642-logging

Conversation

@mbeckerle
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@tuxji tuxji left a comment

Choose a reason for hiding this comment

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

Unit tests are failing, needs attention.

Copy link
Copy Markdown
Member

@stevedlawrence stevedlawrence left a comment

Choose a reason for hiding this comment

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

+1

// work.
val e = new SAXParseException(
s"""DaffodilXMLLoader: Unable to resolve schemaLocation='$systemId'.""", null)
throw e
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this a bug in Xerces where it doesn't consider faling to resolve schemaLocation an error? Or maybe there's some configuration we aren't setting correctly to causes this to be an errors? Seems odd that a failure to resolve this specific thing isn't considered an error when all other failures are handled by Xerces correctly.

Also, I wonder if PR #745, which switches a different xml resolver, would have any affect on this?

@mbeckerle mbeckerle merged commit 9548825 into apache:main Feb 2, 2022
@mbeckerle mbeckerle deleted the daf-2642-logging branch February 2, 2022 13:16
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.

3 participants