[jira] [Closed] (TINKERPOP-848) Support default attribute values in GraphMLReader

["Yang Xia (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/TINKERPOP-848?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Yang Xia closed TINKERPOP-848.
------------------------------
    Resolution: Won't Do

Closing given [discussion|https://lists.apache.org/thread/om2m0phg25s83529p9w0gldmcxz7578h] - it can be reopened if there is expectation that there will be active work on this item.

> Support default attribute values in GraphMLReader
> -------------------------------------------------
>
>                 Key: TINKERPOP-848
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP-848
>             Project: TinkerPop
>          Issue Type: Improvement
>          Components: io
>    Affects Versions: 3.0.2-incubating
>            Reporter: Pavel Klinov
>            Priority: Trivial
>   Original Estimate: 2h
>  Remaining Estimate: 2h
>
> Looking at the code of GraphMLReader I see that it doesn't support default values of attributes, which are allowed by the GraphML spec. This is a bit annoying especially if the input defines default values for attributes which are used for mandatory data, e.g. edge labels. 
> One small example is the sample graph at [1]. "d_e" is the label attribute with a default value. There're <edge .. /> elements w/o body later in the document and reading those will throw a "java.lang.IllegalArgumentException: Label can not be null" exception (if the vendor considers edge labels mandatory).
> I'd personaly squash both keyIdMap and keyTypesMap into a single String -> AttrInfo map, where AttrInfo would contain information about the data attribute name, type, and the default value.
> [1] http://www.eecs.wsu.edu/~yyao/DirectedStudyI/Datasets/AS/sample.graphml



--
This message was sent by Atlassian Jira
(v8.20.10#820010)