-
Notifications
You must be signed in to change notification settings - Fork 3
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
Use schema from schemaLocation URL #72
Conversation
public Set<String> parse(HttpEntity entity) throws IOException | ||
{ | ||
InputStream is = entity.getContent(); | ||
rd = new JsonReader(new InputStreamReader(is)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DefaultCharset: Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations. (details)
(at-me in a reply with help
or ignore
)
protected void saveMissingXsds() throws IOException | ||
{ | ||
File file = new File(outDir, "missing_xsds.txt"); | ||
FileWriter wr = new FileWriter(file); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DefaultCharset: Implicit use of the platform default charset, which can result in differing behaviour between JVM executions or incorrect behavior if the encoding of the data source doesn't match expectations. (details)
(at-me in a reply with help
or ignore
)
*/ | ||
public class SchemaDao | ||
{ | ||
private Logger log; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UnusedVariable: The field 'log' is never read. (details)
(at-me in a reply with help
or ignore
)
Hi @tdddblog , I am testing the schema pull requests and I am having a message with harvest, missing harvest/registry. I am guessing I should have in my configuration the url of the elasticsearch. How should that be done ? Thanks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That works, thanks Eugene
🗒️ Summary
Use schema from schemaLocation URL
(See NASA-PDS/registry-mgr#31)
⚙️ Test Instructions
registry-manager
onschema
branch)missing_fields.txt
andmissing_xsds.txt
files.♻️ Related Issues
NASA-PDS/registry-mgr#31
NASA-PDS/registry-mgr#39