[GitHub] [tinkerpop] spmallette opened a new pull request #1240: TINKERPOP-2312 Empty keys to group() should group to null

[GitBox <gi...@apache.org>]

spmallette opened a new pull request #1240: TINKERPOP-2312 Empty keys to group() should group to null
URL: https://github.com/apache/tinkerpop/pull/1240
 
 
   https://issues.apache.org/jira/browse/TINKERPOP-2312
   
   Basically, this:
   
   ```text
   gremlin> g.V().group().by('age')
   The property does not exist as the key has no associated value for the provided element: v[3]:age
   Type ':help' or ':h' for help.
   Display stack trace? [yN]n
   ```
   
   now does this:
   
   ```text
   gremlin> g.V().groupCount().by('age')
   ==>[null:2,32:1,35:1,27:1,29:1]
   ```
   
   which has its conveniences but also produces a result that would be easier to follow for new users as opposed to the old exception. 
   
   All tests pass with `docker/build.sh -t -n -i`
   
   VOTE +1
   
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [tinkerpop] spmallette merged pull request #1240: TINKERPOP-2312 Empty keys to group() should group to null

[GitBox <gi...@apache.org>]

spmallette merged pull request #1240: TINKERPOP-2312 Empty keys to group() should group to null
URL: https://github.com/apache/tinkerpop/pull/1240
 
 
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services