Github user rjbriody commented on the pull request:
https://github.com/apache/incubator-tinkerpop/pull/242#issuecomment-188858152
1 Done
2
Upgrade Instructions:
The profile()-Step has been refactored into 2 steps - the ProfileStep and
the ProfileSideEffectStep. Users who previously used the profile()-Step in
conjunction with cap(TraversalMetrics.METRICS_KEY) can now simply omit the cap
step. Users who retrieved TraverseralMetrics from the side effects after
iteration can still do so, but will need to specify a side effect key when
using the profile()-Step.
CHANGELOG
Split existing profile()-Step into ProfileStep and ProfileSideEffectStep.
profile()-Step now emits TraversalMetrics without the need for cap()-Step.
3 I'm guessing this is outside the coverage of the tests? I'm not hooked up
to run Giraph or Spark (and don't have time to go down that rabbit hole). I
could definitely use some help here if testing those is required for merge.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
