Marko A. Rodriguez created TINKERPOP3-854:
---------------------------------------------
Summary: Remove PropertyMapStep in favor of multi-instruction
construction.
Key: TINKERPOP3-854
URL: https://issues.apache.org/jira/browse/TINKERPOP3-854
Project: TinkerPop 3
Issue Type: Improvement
Reporter: Marko A. Rodriguez
Fix For: 3.1.0-incubating
{code}
gremlin> g.V().valueMap()
==>[name:[marko], age:[29]]
==>[name:[vadas], age:[27]]
==>[name:[lop], lang:[java]]
==>[name:[josh], age:[32]]
==>[name:[ripple], lang:[java]]
==>[name:[peter], age:[35]]
gremlin> g.V().local(properties().group().by(key).by(value))
==>[name:[marko], age:[29]]
==>[name:[vadas], age:[27]]
==>[name:[lop], lang:[java]]
==>[name:[josh], age:[32]]
==>[name:[ripple], lang:[java]]
==>[name:[peter], age:[35]]
{code}
...and I checked it works for multi-properties.
{code}
gremlin> g.V().propertyMap()
==>[name:[vp[name->marko]], age:[vp[age->29]]]
==>[name:[vp[name->vadas]], age:[vp[age->27]]]
==>[name:[vp[name->lop]], lang:[vp[lang->java]]]
==>[name:[vp[name->josh]], age:[vp[age->32]]]
==>[name:[vp[name->ripple]], lang:[vp[lang->java]]]
==>[name:[vp[name->peter]], age:[vp[age->35]]]
gremlin> g.V().local(properties().group().by(key).by())
==>[name:[vp[name->marko]], age:[vp[age->29]]]
==>[name:[vp[name->vadas]], age:[vp[age->27]]]
==>[name:[vp[name->lop]], lang:[vp[lang->java]]]
==>[name:[vp[name->josh]], age:[vp[age->32]]]
==>[name:[vp[name->ripple]], lang:[vp[lang->java]]]
==>[name:[vp[name->peter]], age:[vp[age->35]]]
{code}
We should do some performance checking and as long as its reasonably the same
speed, drop {{PropertyMapStep}}.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
