- 23 Nov, 2016 1 commit
-
-
Bilal Ahmad committed
-
- 03 Nov, 2016 2 commits
-
-
Add a default response size and a response size limit.
Toby Lawrence committed -
By default, we currently allow a user to retrieve all responses from a thread, which is terrible from a performance standpoint. This change adds a default response size of 100, which was chosen to match typical production pagination parameters, and a response limit of 200. Given that responses can include children comments, 200 is still a large number, but far less disastrous than, say, getting 3000 responses, with all of their children comments. Both of these values are configurable, and can be overriden via environment variables.
Toby Lawrence committed
-
- 29 Sep, 2016 1 commit
-
-
Add method tracing inside user.
Robert Raposa committed
-
- 28 Sep, 2016 1 commit
-
-
Robert Raposa committed
-
- 15 Sep, 2016 1 commit
-
-
TNL-5115: Remove sort order from cs_comment_service
Robert Raposa committed
-
- 13 Sep, 2016 2 commits
-
-
Robert Raposa committed
-
Robert Raposa committed
-
- 12 Sep, 2016 1 commit
-
-
Robert Raposa committed
-
- 09 Sep, 2016 2 commits
-
-
MA-2139: Mark thread as read for leaner on thread/comment creation
wajeeha-khalid committed -
wajeeha-khalid committed
-
- 07 Sep, 2016 6 commits
-
-
Add an OEP-2 compliant openedx.yaml file
Calen Pennington committed -
Calen Pennington committed
-
Calen Pennington committed
-
Add the ability to request a thread without its responses.
Robert Raposa committed -
For cases where we don't need or want the responses on a thread at all, like getting some metadata about a thread, we should be able to request it without the responses. This exists in the ThreadPresenter but didn't exist as an option that could be passed in the API call itself. We've added this as an option now -- with_responses -- that defaults to true, which preserves the existing behavior but does in fact allow toggling. This gives us a backwards-compatible upgrade path to allow changes to be made the LMS so that requests which don't need the responses, or do need the responses, can be explicit about what they want.
Toby Lawrence committed -
MA-2678: replace use of 'updated_at' for 'read' state
wajeeha-khalid committed
-
- 02 Sep, 2016 1 commit
-
-
wajeeha-khalid committed
-
- 24 Aug, 2016 2 commits
-
-
Plucking instead of mapping.
Toby Lawrence committed -
Use pluck instead of map for upvoted/downvoted IDs. Mapping involves doing the projection locally, whereas plucking will ask MongoDB to do the projection for us. This is the difference between sending the whole document back and sending back the ObjectId itself. Multipled against hundreds or thousands of documents, mapping can really add up, so plucking should really help these 99%'ile cases.
Toby Lawrence committed
-
- 23 Aug, 2016 5 commits
-
-
Add more tracing + code cleanup.
Toby Lawrence committed -
Toby Lawrence committed
-
Toby Lawrence committed
-
Toby Lawrence committed
-
Toby Lawrence committed
-
- 22 Aug, 2016 2 commits
-
-
Revert "Merge pull request #196 from edx/platform/upgrade-to-ruby23"
Toby Lawrence committed -
This reverts commit 22178086, reversing changes made to 14ed8a85.
Toby Lawrence committed
-
- 19 Aug, 2016 2 commits
-
-
Upgrade to Ruby 2.3.0
Toby Lawrence committed -
Toby Lawrence committed
-
- 18 Aug, 2016 2 commits
-
-
Use proper relations instead of forcefully loading relations by hand.
Toby Lawrence committed -
Some of the ways comments are accessed in the User model (when a complete user object is requested) lead to an N+1 situation because while we preload the related documents (preloading the parent thread of a given comment, in this case), further accesses to those comments actually never take advantage of the preloaded relations. We should be using the relation directly, which will still act the same as before but allow us to actually benefit from preloading.
Toby Lawrence committed
-
- 11 Aug, 2016 9 commits
-
-
Toby Lawrence committed
-
Wait 10s after starting ElasticSearch.
Toby Lawrence committed -
Now that MongoDB 3.0 seems to download/untar faster than 2.6, we seem to be right over the line of how quickly ES can start up and be listening for connections, which leads the tests to try and connect too quickly, which fails. By sleeping for 10 seconds after starting ElasticSearch, it's almost a certainty it will be up and running by the time we try to first connect to it.
Toby Lawrence committed -
Toby Lawrence committed
-
Toby Lawrence committed
-
Toby Lawrence committed
-
Toby Lawrence committed
-
Toby Lawrence committed
-
Switch to using MongoDB 3.0 in tests.
Toby Lawrence committed
-