murblanc commented on pull request #1684:
URL: https://github.com/apache/lucene-solr/pull/1684#issuecomment-686342727
> ```
> Iterable<PropertyValues> values = someFactory.createPropertyFetcher()
> .withSystemProperty("AvailabilityZone")
> .withCoresCount()
> .fetchFromAllNodes()
> ```
@noblepaul this can indeed lead to a simplification if we always fetch the
same set of data from all nodes (from which we fetch data, doesn't have to be
all nodes in the cluster). I didn't think of it this way and I was trying to
design something more flexible. I like the approach above and indeed it can be
as efficient (and less verbose) than my proposal.
Can we make this assumption? Plugin requests a set of properties from a set
of nodes, and this is done efficiently.
If so I'll gladly simplify my proposal.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org
