tinkerpop-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [tinkerpop] spmallette opened a new pull request #1181: TINKERPOP-2280 Changed toString of T
Date Thu, 15 Aug 2019 20:02:46 GMT
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

Mime
View raw message