Implement bulk operations on the split modulestore.

John, I'm the primary culprit on this intermixing as my IDE automatically
adds sphinx docstrings. I should fix the IDE, but I've been neglectful or
too busy.


On Fri, Aug 22, 2014 at 10:11 AM, John Eskew <notifications@github.com>
wrote:

> In common/lib/xmodule/xmodule/modulestore/split_mongo/mongo_connection.py:
>
> >          """
> > -        Find the course_index matching the query. Right now the query must be a legal mongo query
> > -        :param query: a mongo-style query of {key: [value|{$in ..}|..], ..}
> > +        Find the course_index matching particular conditions.
> > +
> > +        :param branch: If specified, this branch must exist in the returned courses
> > +        :param search_targets: If specified, this must be a dictionary specifying field values
> > +            that must exist in the search_targets of the returned courses
>
> What should be our policy towards the mix of Sphinx-style and Google-style
> docstrings in the code base? Since this docstring is significantly
> modified, shouldn't it match the Google-style docstring added above?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/edx/edx-platform/pull/4780/files#r16599334>.
>