-
Notifications
You must be signed in to change notification settings - Fork 0
xblock and course addressing
EdX code throughout the stack currently treats xblock and course ids as transparent, introspectable, and frequently manipulated addresses with a known set of fields. The applications serialize and deserialize these ids as parsable urls with fixed syntax (org, partial course id, category, block id or org, partial course id, course unique id). This treatment makes it almost impossible to add semantics such as versioning, subordinate organizations, lifecycle snapshots (e.g., draft, review, alpha, beta, published), or course encapsulation to the applications.
The db rearchitecture task known as "Split Mongo" uses all of the additional semantics listed above as well as separating the identity of courses and reusable definitions from the identity of xblocks within courses (usages). Xblock also distinguishes usages from definitions; so, this change is not unilateral. LMS has also always informally separated course identity from xblock usage identity; however, without treating the course id as an object. Split makes these distinctions formal.
We had a plan to have the two id representations exist simultaneously with continuous translation between them as necessary (Rollout Options); however, this plan raised significant performance questions as well as risks around fragility (possibly not finding some hardcoded ids and translating them to the proper representation for the caller or receiver.)
Discussion with Cale, Rob, Ned, DB, and others drove home the point that only the persistence layer should care about xblock usage and definition ids and only the persistence, auth, and registration layers should care about course ids. The application layers (LMS, CMS, analytics, ORA, Forums) should treat these ids as opaque tokens used to perform data CRUD and reference. The application layers should not parse the ids and should not request semantic information from the ids (e.g., category, org, course_id).
All edX apps currently serialize and deserialize the ids as subfields from urls; thus, to make the apps able to handle any persistence layer's id representations, we will need to change all of the urls and the url parsing. Nevertheless, we must provide deprecated backward compatibility which can still interpret hardcoded and bookmarked urls using the existing syntax.
The crux of the proposed solution is that all ids (course, xblock, definition) will be opaque keys to the applications. The persistence, auth, and enrollment modules may introspect these keys as necessary for additional information. If an application needs additional information such as org, course_id, category and just has an id, the application must ask the owning service.
No application should assume such requests are free: that is, the service may perform db operations or other non-trivial lookups to answer the query. Sometimes context will make it clear what behaviors the service will likely support for the id (e.g., course_id for an xblock usage but not for an xblock definition, org for a course_id but not necessarily for a definition, category for an xblock usage or definition but not for a course id) but some services may support behaviors which other services don't support (e.g., version and version history for an xblock usage, list of available lifecycle snapshots such as draft v preview v live). Xblocks will themselves support some of these behaviors and, so, in many cases, it may make more sense for the app to retrieve the xblock given the address and query it.
A corollary of this change is that no application should assume the serialized (url, id, etc) form of an id has any particular parsable fields but only that the id is serialized and deserialized as a single field. THUS, the serialized form will not have any slashes in it so that url parsing can digest the whole id as a single field.
Key
common superclass representing an address or id of something.
-
unicode(Key)
produces a string representation which will have no slashes, question marks, nor ampersands but may have spaces and other non url safe characters for the givenKey
. NOTE, rather than storing a much more restricted stringified version of the key in an htmlid
attr, store this string or the url below in thedata-id
or any otherdata-xxx
field which allows any string chars unlike html attrs which gravely restrict the char set. -
Key.url()
produces a url safe string for the key which will have no url tag (that is, noi4x://
), slashes, ampersands, nor question marks and is safe to use as a field in a url. -
Key(unicode)
constructs aKey
from theunicode
. The following must be true:Key(unicode(key)) == key
. -
Key.from_url(url)
constructs aKey
from the url. The following must be true:Key.from_url(key.url()) == key
.
Key
is an abstract class and cannot be instantiated. Keys will have namespaces (type indicators) which Key uses to figure out which concrete subclass to instantiate; however, no application should ever try to interpret the namespace indicator nor the payload.
Key
concrete classes register themselves and their unique namespace id with the Key
class. The namespaces, of course, cannot collide. If they do, then Key
may arbitrarily choose which concrete class it uses. (or should it raise an Exception?)
To do more than just serialize and deserialize a key, apps will need to send keys to services which will answer reasonable queries about the keys. Such queries may include org
, category
, course_id
for keys which are not CourseKey
, version
, branch
, etc. Key Services define concrete Key
classes which implement the Key
class interface and register their namespaces with Key
.
We will define two Key services; however, any persistence layer may add any other Key services.
LocationService
is a Key
service roughly corresponding to our existing Location
type. It defines CourseId
and XblockLocation
classes and accepts instances of these to answer queries such as category
and org
.
LocatorService
is a Key
service which handles the Locator
concrete classes.
CourseKey
is an abstract subclass of Key
to represent keys which are course ids. These should add support for the following:
-
course_id
is a property of aCourseKey
instance which returns the unique id for a given course offering. Note, this is not acourse
xblock but instead the offering to which students register, instructors add staff, and SplitMongo provides indices. Acourse_id
is a unicode string which uniquely identifies the course. It must obey the same syntax rules asKey.url()
. -
CourseKey.from_course_id(course_id)
is aCourseKey
constructor which inverts thecourse_id
property with the concomitant equality requirements. -
org
? I'm concerned that org is actually ambiguous.harvard
is an org, but so areharvard.humanities
andharvard.humanities.political_science
; however, we could defineorg
as a
CourseId
is a concrete implementation of CourseKey
which represents edX's traditional org, partial course id, and run id triple. Applications should never assume that a CourseKey
is or will be a CourseId
and thus should not depend on these 3 fields having any meaning with respect to a CourseKey
. The LocationService
defines and supports the CourseId
class. NOTE, no application should treat course ids as strings of triples any more.
CourseLocator
is the existing CourseLocator
class. LocatorService
defines and supports this class.
In addition to the above properties and functions, this supports the following which only the persistence layer should count on.
version