You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tinkerpop.apache.org by "vkagamlyk (via GitHub)" <gi...@apache.org> on 2023/02/18 00:45:31 UTC

[GitHub] [tinkerpop] vkagamlyk commented on pull request #1843: TINKERPOP-2824 Properties on response elements

vkagamlyk commented on PR #1843:
URL: https://github.com/apache/tinkerpop/pull/1843#issuecomment-1435427556

   > Great change! I just added a few comments but basically all of them are only nitpicks.
   > 
   > Apart from that, I don't see any documentation for this new functionality. I think this needs to be documented in a lot of places (upgrade docs, changelog, reference docs, I/O docs). We should also definitely document that this changes the default behavior which will now include properties for all returned vertices and edges.
   > 
   > Do you want to do that in a follow-up PR?
   
   I plan to add documentation update to this PR, I first wanted the PR to be more or less formed


-- 
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: commits-unsubscribe@tinkerpop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org