Skip to content

Move to a different Json Schema validator - #2222

Merged
lossyrob merged 1 commit into
locationtech:masterfrom
pomadchin:feature/json-validation
Jun 2, 2017
Merged

lossyrob merged 1 commit into
locationtech:masterfrom
pomadchin:feature/json-validation

Conversation

@pomadchin

@pomadchin pomadchin commented May 31, 2017 •

Copy link
Copy Markdown
Member

A lightweight https://github.com/networknt/json-schema-validator is used instead of https://github.com/java-json-tools/json-schema-validator.

Were applied some changes into the input schema definition (it was incorrect).
Some changes into printing validation error messages were added (logger used, error messages are red)

A source with tests for different json schema validators in different languages:
https://github.com/json-schema-org/JSON-Schema-Test-Suite

@pomadchin
pomadchin requested a review from lossyrob May 31, 2017 22:48
@pomadchin
pomadchin force-pushed the feature/json-validation branch from 41fc405 to f1f9b8b Compare May 31, 2017 22:48
@pomadchin
pomadchin force-pushed the feature/json-validation branch from f1f9b8b to a9ab63d Compare May 31, 2017 22:51
@pomadchin
pomadchin force-pushed the feature/json-validation branch from a9ab63d to e732358 Compare May 31, 2017 22:52
@lossyrob lossyrob added this to the 1.1 milestone Jun 1, 2017
@lossyrob

lossyrob commented Jun 2, 2017 •

Copy link
Copy Markdown
Member

Dependency Additions

json-schema-validator-dependency-graph

CQs required for

Json Schema Validator

  • com.networknt:json-schema-validator:0.1.7

Jackson

  • com.fasterxml.jackson.core:jackson-annotations:2.8.0
  • com.fasterxml.jackson.core:jackson-core:2.8.7
  • com.fasterxml.jackson.core:jackson-databind:2.8.7

Question: Why is jackson-annotations at 2.8.0 for the 2.8.7 versions? Can we submit these three as a single CQ (do they share the same repository?)

Note there are approved CQs for jackson 1.5.0

Apache Commons Lang

  • org.apache.commons:commons-lang3:3.5

3.5 should be submitted, Apache commons-lang3 version 3.4 is already approved.

@pomadchin

pomadchin commented Jun 2, 2017 •

Copy link
Copy Markdown
Member Author

I decided to draw the whole project graph too using

./sbt spark-etl/dependencyGraphMl

Schema validator neighbours:

image

This information makes sense, as it corresponds to com.networknt:json-schema-validator:0.1.7 (approved, CQ 13652) pom.xml

But annotations version is a mystery.

image

Looks like it really should be 2.8.0 (probably their internal versions inconsistency): https://mvnrepository.com/artifact/com.fasterxml.jackson.core/jackson-databind/2.8.7

Requests should be sent in all cases (same version / different versions of these deps) separately:

org.apache.commons:commons-lang3:3.5 (approved, CQ 13653) neighbours:

image

At least opencsv depends on it too, request should be done too.

There is also org.apache.commons:commons-lang3:3.1 in deps:

image

But it's evicted by 3.5

Update: all deps are approved.

@lossyrob

lossyrob commented Jun 2, 2017

Copy link
Copy Markdown
Member

CQ's Approved. +1

@lossyrob
lossyrob merged commit 936460f into locationtech:master Jun 2, 2017
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