You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by GitBox <gi...@apache.org> on 2022/01/12 01:11:12 UTC

[GitHub] [tinkerpop] bechbd opened a new pull request #1546: Squashed all commits and merged from master

bechbd opened a new pull request #1546:
URL: https://github.com/apache/tinkerpop/pull/1546


   This pull request extends the `property()` step to take a `Map` argument by adding two new signatures:
   
   ```
   property(Map)
   property(Cardinality, Map)
   ```
   When called, each individual key/value pair in the map is saved as a property to the element.
   When the cardinality is specified, that cardinality will be applied to all elements in the map as they are saved to the element.
   
   ```
   gremlin> g.addV('person').property(['first_name': 'Dave', 'last_name': 'B']).valueMap()
   ==>[last_name:[B],first_name:[Dave]]
   gremlin> g.addV('person').property(single, ['first_name': 'Dave', 'last_name': 'B']).valueMap()
   ==>[last_name:[B],first_name:[Dave]]
   ```
   
   Known limitations
   If users need different cardinalities per property, then please use the existing pattern of stringing multiple property() calls together.


-- 
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: commits-unsubscribe@tinkerpop.apache.org

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

[GitHub] [tinkerpop] spmallette commented on pull request #1546: Added ability to specify a Map for property() step

Posted by GitBox <gi...@apache.org>.
spmallette commented on pull request #1546:
URL: https://github.com/apache/tinkerpop/pull/1546#issuecomment-1013060157


   I think this looks good now - code review was on #1533 - merging.


-- 
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: commits-unsubscribe@tinkerpop.apache.org

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

[GitHub] [tinkerpop] spmallette merged pull request #1546: Added ability to specify a Map for property() step

Posted by GitBox <gi...@apache.org>.
spmallette merged pull request #1546:
URL: https://github.com/apache/tinkerpop/pull/1546


   


-- 
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: commits-unsubscribe@tinkerpop.apache.org

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