[
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.
(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.)
> 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.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)