[GitHub] [incubator-hugegraph-toolchain] coderzc opened a new pull request, #376: [fix] Fix deserialize json

[GitBox]

coderzc opened a new pull request, #376:
URL: https://github.com/apache/incubator-hugegraph-toolchain/pull/376

   ### Motivation
   
   ```java
   SLF4J: Class path contains multiple SLF4J bindings.
   SLF4J: Found binding in 
[jar:file:/Users/zc/IdeaProjects/incubator-hugegraph-toolchain/hugegraph-loader/hugegraph-loader-1.0.0/lib/log4j-slf4j-impl-2.18.0.jar!/org/slf4j/impl/StaticLoggerBinder.class]
   SLF4J: Found binding in 
[jar:file:/Users/zc/IdeaProjects/incubator-hugegraph-toolchain/hugegraph-loader/hugegraph-loader-1.0.0/lib/slf4j-log4j12-1.6.1.jar!/org/slf4j/impl/StaticLoggerBinder.class]
   SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an 
explanation.
   SLF4J: Actual binding is of type 
[org.apache.logging.slf4j.Log4jLoggerFactory]
   
   Failed to start loading, cause: Failed to deserialize json
   Exception in thread "main" org.apache.hugegraph.rest.SerializeException: 
Failed to deserialize json
        at org.apache.hugegraph.loader.util.JsonUtil.fromJson(JsonUtil.java:77)
        at 
org.apache.hugegraph.loader.util.MappingUtil.parse(MappingUtil.java:84)
        at 
org.apache.hugegraph.loader.mapping.LoadMapping.of(LoadMapping.java:68)
        at 
org.apache.hugegraph.loader.HugeGraphLoader.<init>(HugeGraphLoader.java:80)
        at 
org.apache.hugegraph.loader.HugeGraphLoader.<init>(HugeGraphLoader.java:76)
        at 
org.apache.hugegraph.loader.HugeGraphLoader.main(HugeGraphLoader.java:67)
   Caused by: com.fasterxml.jackson.databind.exc.InvalidDefinitionException: 
Conflicting property-based creators: already had explicitly marked creator 
[constructor for org.apache.hugegraph.loader.mapping.LoadMapping, annotations: 
{interface 
com.fasterxml.jackson.annotation.JsonCreator=@com.fasterxml.jackson.annotation.JsonCreator(mode=DEFAULT)}],
 encountered another: [constructor for 
org.apache.hugegraph.loader.mapping.LoadMapping, annotations: {interface 
com.fasterxml.jackson.annotation.JsonCreator=@com.fasterxml.jackson.annotation.JsonCreator(mode=DEFAULT)}]
    at [Source: (String)"{
     "version": "2.0",
     "structs": [
       {
         "id": "1",
         "skip": false,
         "input": {
           "type": "FILE",
           "path": 
"computer-dist/src/assembly/dataset/ml-latest-small/ratings.csv",
           "file_filter": {
             "extensions": [
               "*"
             ]
           },
           "format": "CSV",
           "delimiter": ",",
           "date_format": "yyyy-MM-dd HH:mm:ss",
           "time_zone": "GMT+8",
           "skipped_line": {
             "regex": "(^#|^//).*|"
           },
           ""[truncated 4213 chars]; line: 1, column: 1]
        at 
com.fasterxml.jackson.databind.exc.InvalidDefinitionException.from(InvalidDefinitionException.java:62)
        at 
com.fasterxml.jackson.databind.deser.BeanDeserializerFactory.buildBeanDeserializer(BeanDeserializerFactory.java:227)
        at 
com.fasterxml.jackson.databind.deser.BeanDeserializerFactory.createBeanDeserializer(BeanDeserializerFactory.java:143)
        at 
com.fasterxml.jackson.databind.deser.DeserializerCache._createDeserializer2(DeserializerCache.java:414)
        at 
com.fasterxml.jackson.databind.deser.DeserializerCache._createDeserializer(DeserializerCache.java:349)
        at 
com.fasterxml.jackson.databind.deser.DeserializerCache._createAndCache2(DeserializerCache.java:264)
        at 
com.fasterxml.jackson.databind.deser.DeserializerCache._createAndCacheValueDeserializer(DeserializerCache.java:244)
        at 
com.fasterxml.jackson.databind.deser.DeserializerCache.findValueDeserializer(DeserializerCache.java:142)
        at 
com.fasterxml.jackson.databind.DeserializationContext.findRootValueDeserializer(DeserializationContext.java:479)
        at 
com.fasterxml.jackson.databind.ObjectMapper._findRootDeserializer(ObjectMapper.java:4405)
        at 
com.fasterxml.jackson.databind.ObjectMapper._readMapAndClose(ObjectMapper.java:4214)
        at 
com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3214)
        at 
com.fasterxml.jackson.databind.ObjectMapper.readValue(ObjectMapper.java:3182)
        at org.apache.hugegraph.loader.util.JsonUtil.fromJson(JsonUtil.java:74)
        ... 5 more
   ```
   
   ### Modifications
   
   * Remove conflicted `@JsonCreator`.
   * Add mapping v2 test.
   * Throw Exception in the main method.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@hugegraph.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org