spmallette opened a new pull request, #3667:
URL: https://github.com/apache/tinkerpop/pull/3667
The support-check example referenced a traversal source g that was only
defined in an earlier, separate code block. It ran without error solely because
TinkerGraph does not support transactions, so the ternary's true branch was
never evaluated. A reader adapting this pattern for a transactional graph would
have hit an error on the undefined g. Define g from graph within the block so
the example stands on its own. Also add a short note describing how the
features() output is grouped, so a reader can orient in the long list of
boolean flags.
<!--
Thanks for contributing! Reminders:
+ TARGET the earliest branch where you want the change
3.7-dev -> 3.7.7 (non-breaking only)
3.8-dev -> 3.8.2 (non-breaking only)
master -> 4.0.0
+ Committers will MERGE the PR forward to newer versions
+ ADD entry to the CHANGELOG.asciidoc for the targeted version
Do not reference a JIRA number there
+ ADD JIRA number to title and link in description
+ PRs requires 3 +1s from committers OR
1 +1 and 7 day wait to merge.
+ MORE details: https://s.apache.org/rtnal
-->
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]