We just received this PR from Mathias Bogaert who is working on a Graph
implementation:

https://github.com/apache/incubator-tinkerpop/pull/315

It removes dashes from keys used in our test suite.  The nature of those
tests is not to enforce that Graphs can support dashes (dashes were just
arbitrarily used in the tests), so I'm inclined to support this change.
Does anyone feel differently?  If so, I think the recommendation would be
to specifically create tests that "enforce dashes in key names".

If there are no additional comments on this in the next 72 hours (Friday,
April 20, 2016, 6:45am EST) I will assume lazy consensus and merge via CTR.

Reply via email to