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.

Sign up for GitHub

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

Jump to bottom

Reduced number of queries when object name case differs accross rollups : Multiple LRE Contexts when rollup definition only differs by case #229

Closed
jondavis9898 opened this issue Aug 19, 2015 · 1 comment
Closed

Reduced number of queries when object name case differs accross rollups : Multiple LRE Contexts when rollup definition only differs by case #229

jondavis9898 opened this issue Aug 19, 2015 · 1 comment
Labels
bug ready for next release

Comments

@jondavis9898
Copy link
Contributor

When defining a rollup, if the only difference in the definition is the case (e.g. abc vs ABC) of a value specified for parent object, relationshipfield or order by, multiple LRE Contexts are created resulting in multiple SOQL statements.

For example, if two rollups are exactly the same but differ only by their "Parent Object" value of "LookupParent__c" and "lookupparent__c", a single context could be used but two contexts will be created and processed and two identical SOQL statements issued.
jondavis9898 added a commit to jondavis9898/declarative-lookup-rollup-summaries that referenced this issue Aug 20, 2015
@jondavis9898
issue SFDO-Community#229 - tests to demonstrate issue
e42deb1 
Note that this commit contains extra debug statements to make observing
actual vs expected result easier.  The debug statements will be removed
if this issue is determined to be valid and PR accepted.
jondavis9898 added a commit to jondavis9898/declarative-lookup-rollup-summaries that referenced this issue Aug 20, 2015
@jondavis9898
issue SFDO-Community#229 - Ensure context is shared when differ only …
60d8e3d 
…by case

Additional debug statements have been removed from tests in this commit.
jondavis9898 added a commit to jondavis9898/declarative-lookup-rollup-summaries that referenced this issue Aug 20, 2015
@jondavis9898
Issue SFDO-Community#229 - tests to demonstrate issue
1633b43 
Note that this commit contains extra debug statements to make observing
actual vs expected result easier.  The debug statements will be removed
if this issue is determined to be valid and PR accepted.
jondavis9898 added a commit to jondavis9898/declarative-lookup-rollup-summaries that referenced this issue Aug 20, 2015
@jondavis9898
Issue SFDO-Community#229 - Ensure context is shared when rollups diff…
023ae41 
…er by case only

Additional debug statements have been removed from tests in this commit.
jondavis9898 added a commit to jondavis9898/declarative-lookup-rollup-summaries that referenced this issue Aug 20, 2015
@jondavis9898
Issue SFDO-Community#229 - Improve test coverage for lookuprollupsumm…
f0491ce 
…ary__c update
@afawcett afawcett added the bug label Aug 27, 2015
afawcett added a commit that referenced this issue Aug 27, 2015
@afawcett
Merge pull request #231 from jondavis9898/issue229-extra-contexts-whe…
80406ca 
…n-case-different

Issue #229 - Ensure context is shared when rollups differ by case only
@afawcett afawcett changed the title Multiple LRE Contexts when rollup definition only differs by case Reduced number of queries when object name case differs accross rollups : Multiple LRE Contexts when rollup definition only differs by case Aug 30, 2015
@afawcett
Copy link
Collaborator

Fixed in v1.23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug ready for next release
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@afawcett @jondavis9898