[
https://issues.apache.org/jira/browse/TINKERPOP-3032?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Stephen Mallette updated TINKERPOP-3032:
----------------------------------------
Description: gremlin-language doesn't have {{Set}} yet until TINKERPOP-3023
but .NET forces {{PartitionStrategy.readPartitions}} to be of that type. The
other languages accept {{List}}. There should be some alignment here. {{Set}}
makes sense for this configuration but from an API perspective users probably
don't need to be forced into it. Seems like it could internally use {{Set}} but
accept a {{List}} to avoid that extra specificity. This also applies to Go.
{{Set}} is also used in constructing some other strategies like
{{ReservedKeysVerificationStrategy}}. (was: gremlin-language doesn't have
{{Set}} yet until TINKERPOP-3023 but .NET forces
{{PartitionStrategy.readPartitions}} to be of that type. The other languages
accept {{List}}. There should be some alignment here. {{Set}} makes sense for
this configuration but from an API perspective users probably don't need to be
forced into it. Seems like it could internally use {{Set}} but accept a
{{List}} to avoid that extra specificity. This also applies to Go. )
> Align readPartitions data type for PartitionStrategy
> ----------------------------------------------------
>
> Key: TINKERPOP-3032
> URL: https://issues.apache.org/jira/browse/TINKERPOP-3032
> Project: TinkerPop
> Issue Type: Improvement
> Components: dotnet, go, process
> Affects Versions: 3.6.6
> Reporter: Stephen Mallette
> Priority: Minor
>
> gremlin-language doesn't have {{Set}} yet until TINKERPOP-3023 but .NET
> forces {{PartitionStrategy.readPartitions}} to be of that type. The other
> languages accept {{List}}. There should be some alignment here. {{Set}} makes
> sense for this configuration but from an API perspective users probably don't
> need to be forced into it. Seems like it could internally use {{Set}} but
> accept a {{List}} to avoid that extra specificity. This also applies to Go.
> {{Set}} is also used in constructing some other strategies like
> {{ReservedKeysVerificationStrategy}}.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
