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 2019/08/15 20:02:46 UTC

[GitHub] [tinkerpop] spmallette opened a new pull request #1181: TINKERPOP-2280 Changed toString of T

spmallette opened a new pull request #1181: TINKERPOP-2280 Changed toString of T
URL: https://github.com/apache/tinkerpop/pull/1181
 
 
   https://issues.apache.org/jira/browse/TINKERPOP-2280
   
   Reduce confusion when `T` is in a `Map` and users overload property keys that match `T` instance names like "label" and "id". I think I'm still classifying this as a non-breaking change despite changing the form of the `toString()`. Folks probably shouldn't be relying on that in their code. I would think they would simply use `name()` as you typically would with an `enum` if you were working with the string representation. 
   
   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