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

mlLoadSchemas does not work with folders #160

Closed
SoundBot opened this issue Feb 10, 2017 · 6 comments
Closed

mlLoadSchemas does not work with folders #160

SoundBot opened this issue Feb 10, 2017 · 6 comments
Milestone

Comments

@SoundBot
Copy link

Steps to reproduce:

  1. Create ml-schemas folder
  2. Create another folder with xsd inside ml-schemas
  3. Run mlLoadSchemas

The load fails.
It works fine when all XSDs are located in ml-schemas folder.

@rjrudin
Copy link
Contributor

rjrudin commented Feb 12, 2017

Thanks @SoundBot , that's certainly the case - the initial design for this feature was simply to load TDE schemas from a flat directory. Adding this should be straightforward.

@rjrudin
Copy link
Contributor

rjrudin commented Feb 20, 2017

Going to address this in 2.7.0, which I'll hopefully release in the next couple weeks. Going to also address how to load redaction rulesets with custom collections. Most of the work will be in ml-javaclient-util.

@rjrudin
Copy link
Contributor

rjrudin commented Feb 22, 2017

@SoundBot Any issues if schemas are now prefixed with a "/", just like modules?

@rjrudin rjrudin closed this as completed Feb 22, 2017
@jeff-goddard
Copy link

Any issues if schemas are now prefixed with a "/", just like modules?
Is there any way to revert to the previous schema URI without a '/' prefix in the URI?

@rjrudin
Copy link
Contributor

rjrudin commented Oct 18, 2017

@jedigo There is, though it's not available yet via a Gradle property. The code is all in the ml-javaclient-util project. The class that reads files from the filesystem is DefaultDocumentFileReader. It has a setUriPrefix method to control the URI prefix, which defaults to "/". It's possible to do some surgery in build.gradle to access this class and modify it, but a Gradle property of course would be nicer. If that sounds useful, please open a new ticket for it.

rjrudin pushed a commit that referenced this issue Aug 23, 2024
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

No branches or pull requests

3 participants