Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

executeOnEntries in callable ignores index #1156

Closed
mraad opened this issue Nov 17, 2013 · 3 comments

Comments

Projects
None yet
7 participants
@mraad
Copy link

commented Nov 17, 2013

When a task is submitted to all members, and in the call an executeOnEntries is invokes with a predicate that is based on an index - the index is ignored and a "full scan" of the "local" members is performed.

@eminn

This comment has been minimized.

Copy link
Collaborator

commented Nov 17, 2013

You are right about this. It should not make a full table scan. I remember
while the feature implemented in the first place the decision made
with considering the backward compatibility. I'll discuss it with the team
if we can implement it in a more performant way without breaking the
compatibility.

17 Kasým 2013 Pazar tarihinde Mansour adlý kullanýcý þöyle yazdý:

When a task is submitted to all members, and in the call an
executeOnEntries is invokes with a predicate that is based on an index -
the index is ignored and a "full scan" of the "local" members is performed.

Reply to this email directly or view it on GitHubhttps://github.com//issues/1156
.

@wimdeblauwe

This comment has been minimized.

Copy link

commented Sep 9, 2015

+1 for this

@jerrinot jerrinot modified the milestones: 3.7, Backlog Oct 23, 2015

LoneRifle added a commit to LoneRifle/hazelcast that referenced this issue Apr 12, 2016

Port 0b15cc1 from @torkeld to fix hazelcast#1156 and hazelcast#1719
* If the PartitionIteratingOperation has a
PartitionWideEntryWithPredicateOperationFactory, try to query the
indices associated with the map that the factory is scoped for.
* In the factory itself, if the index service returns keys given a
predicate query, retain them and issue a MultipleEntryOperation
containing those keys instead of a
PartitionWideEntryWithPredicateOperation.

LoneRifle added a commit to LoneRifle/hazelcast that referenced this issue Apr 13, 2016

Port torkeld/hazelcast@0b15cc1 from @torkeld to fix hazelcast#1156 and
…hazelcast#1719

* If the PartitionIteratingOperation has a
PartitionWideEntryWithPredicateOperationFactory, try to query the
indices associated with the map that the factory is scoped for.
* In the factory itself, if the index service returns keys given a
predicate query, retain them and issue a MultipleEntryOperation
containing those keys instead of a
PartitionWideEntryWithPredicateOperation.

LoneRifle added a commit to LoneRifle/hazelcast that referenced this issue Apr 13, 2016

Port torkeld/hazelcast@0b15cc1 from @torkeld to fix hazelcast#1156 and
…hazelcast#1719

* If the PartitionIteratingOperation has a
PartitionWideEntryWithPredicateOperationFactory, try to query the
indices associated with the map that the factory is scoped for.
* In the factory itself, if the index service returns keys given a
predicate query, retain them and issue a MultipleEntryOperation
containing those keys instead of a
PartitionWideEntryWithPredicateOperation.

LoneRifle added a commit to LoneRifle/hazelcast that referenced this issue Apr 29, 2016

Port torkeld/hazelcast@0b15cc1 from @torkeld to fix hazelcast#1156 and
…hazelcast#1719

* If the PartitionIteratingOperation has a
PartitionWideEntryWithPredicateOperationFactory, try to query the
indices associated with the map that the factory is scoped for.
* In the factory itself, if the index service returns keys given a
predicate query, retain them and issue a MultipleEntryOperation
containing those keys instead of a
PartitionWideEntryWithPredicateOperation.
@ahmetmircik

This comment has been minimized.

Copy link
Member

commented May 20, 2016

should be fixed via #8053

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.