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

500 update actions is the maximum per one request #21

Closed
heshamMassoud opened this issue May 3, 2017 · 0 comments
Closed

500 update actions is the maximum per one request #21

heshamMassoud opened this issue May 3, 2017 · 0 comments
Labels
commons enhancement ⬆️ important ⭐
Milestone
1.0.0-M6

Comments

@heshamMassoud
Copy link
Contributor

Need to make sure that update actions are less than 500. Should be chunked.
@heshamMassoud heshamMassoud mentioned this issue May 3, 2017
Merged
@heshamMassoud heshamMassoud added this to the backlog milestone Sep 29, 2017
@heshamMassoud heshamMassoud modified the milestones: backlog, 1.0.0-M6 Nov 21, 2017
heshamMassoud added a commit that referenced this issue Nov 30, 2017
@heshamMassoud
#21: Move IT to correct position.
6a4c737
heshamMassoud added a commit that referenced this issue Nov 30, 2017
@heshamMassoud
#21: Add IT for making update request with more than 500 update actions.
2548643
heshamMassoud added a commit that referenced this issue Nov 30, 2017
@heshamMassoud
#21: Rename batchDrafts to batchElements.
8264e71
heshamMassoud added a commit that referenced this issue Nov 30, 2017
@heshamMassoud
#21: Implement batching for product updates.
94456ff
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Make assertions on counters rather than error message.
3cb4fe6
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Change exception to cause since now the error will be resultant …
7aa53b1 
…from thenCompose in batching.
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Fix unit tests and add a unit test for case of actual exception.
b656164
@heshamMassoud heshamMassoud mentioned this issue Dec 4, 2017
Merged
4 tasks
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Add JavaDocs.
012a763
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Add release notes entry.
6aaaa9b
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Use more specific assertions.
e6c2f94
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Replace recursion with normal for.
604de5d
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Fix test names.
a4ed2c5
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Add unit test to cover uniform and non uniform batch splitting.
b53af31
heshamMassoud added a commit that referenced this issue Dec 4, 2017
@heshamMassoud
#21: Fix variable names.
75b1de4
heshamMassoud added a commit that referenced this issue Dec 5, 2017
@heshamMassoud
#21: Add assertion on batch splitting.
bf339f7
heshamMassoud added a commit that referenced this issue Dec 5, 2017
@heshamMassoud
#21: Fix comment.
27349b0
heshamMassoud added a commit that referenced this issue Dec 5, 2017
@heshamMassoud
#21: Refactor IT for update actions batching.
af9e536
heshamMassoud added a commit that referenced this issue Dec 5, 2017
@heshamMassoud
#21: Remove version assertion which is not very safe, since there are…
808bdf2 
… background version changes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
commons enhancement ⬆️ important ⭐
Projects
None yet
Milestone
1.0.0-M6
Development

No branches or pull requests
1 participant
@heshamMassoud