spmallette commented on a change in pull request #986: TINKERPOP-2059 Modulation of valueMap() URL: https://github.com/apache/tinkerpop/pull/986#discussion_r230964542
########## File path: gremlin-test/features/map/AddVertex.feature ########## @@ -336,7 +336,7 @@ Feature: Step - addV() | marko | And the graph should return 2 for count of "g.V().has(\"name\",\"marko\")" - Scenario: g_V_asXaX_hasXname_markoX_outXcreatedX_asXbX_addVXselectXaX_labelX_propertyXtest_selectXbX_labelX_valueMapXtrueX + Scenario: g_V_asXaX_hasXname_markoX_outXcreatedX_asXbX_addVXselectXaX_labelX_propertyXtest_selectXbX_labelX_valueMap_withXtokensX Review comment: ok - as long as we at least exercise the old stuff sufficiently then no need to bring them all back. I just saw these removed and didn't realize that there were others still hanging about. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services