[jira] [Commented] (TINKERPOP3-844) PropertyMapStep should reuse PropertiesStep

["Matthias Broecheler (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TINKERPOP3-844?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14746860#comment-14746860 ] 

Matthias Broecheler commented on TINKERPOP3-844:
------------------------------------------------

This would be incredibly useful to have for implementations like Titan that use a custom query optimizer to speed up traversals.

> PropertyMapStep should reuse PropertiesStep
> -------------------------------------------
>
>                 Key: TINKERPOP3-844
>                 URL: https://issues.apache.org/jira/browse/TINKERPOP3-844
>             Project: TinkerPop 3
>          Issue Type: Improvement
>            Reporter: Matthias Broecheler
>
> Currently, there are two steps which retrieve properties: PropertiesStep and PropertyMapStep. For any vendor implementation wishing to optimize property retrieval this means twice the (redundant) work because each of the steps needs to be extended individually.
> However, the functionality is very similar. PropertyMapStep should be replaced by two steps: PropertiesStep which returns an iterator over properties and a MapAggregatorStep (not tied to the name) which consumes properties from the previous step and builds the map (either with property or values as the the map value) until the element of a property differs.
> This reuse of PropertiesStep would have increase code reuse inside TinkerPop and should not have a significant performance impact. Most importantly, it makes life a lot easier for implementors.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)