You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tinkerpop.apache.org by "DKZed (via GitHub)" <gi...@apache.org> on 2024/11/08 00:03:44 UTC
[PR] Changed structure array tests to use ArrayList to match serialization [tinkerpop]
DKZed opened a new pull request, #2888:
URL: https://github.com/apache/tinkerpop/pull/2888
Two fixes
1. Changed the structure tests regarding array features to use ArrayList as this matches the input of lists resulting from serialization
2. The parameter including the `value` of the test made it so that these tests could not be opted out in a reasonable manner using `specific`. You would have to do something like `specific = shouldSetValueOnEdge[supportsIntegerArrayValues([I@1d2b8249)]`
--
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
Re: [PR] Changed structure feature test parameter name [tinkerpop]
Posted by "lyndonbauto (via GitHub)" <gi...@apache.org>.
lyndonbauto commented on PR #2888:
URL: https://github.com/apache/tinkerpop/pull/2888#issuecomment-2463554494
VOTE+1
--
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
Re: [PR] Changed structure feature test parameter name [tinkerpop]
Posted by "vkagamlyk (via GitHub)" <gi...@apache.org>.
vkagamlyk commented on PR #2888:
URL: https://github.com/apache/tinkerpop/pull/2888#issuecomment-2471540664
VOTE+1
--
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
Re: [PR] Changed structure feature test parameter name [tinkerpop]
Posted by "DKZed (via GitHub)" <gi...@apache.org>.
DKZed commented on PR #2888:
URL: https://github.com/apache/tinkerpop/pull/2888#issuecomment-2471542502
VOTE+1
--
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
Re: [PR] Changed structure feature test parameter name [tinkerpop]
Posted by "DKZed (via GitHub)" <gi...@apache.org>.
DKZed merged PR #2888:
URL: https://github.com/apache/tinkerpop/pull/2888
--
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