Helpers for asynchronous queries

RESOLVED WONTFIX

Status

Cloud Services
Firefox Sync: Backend
RESOLVED WONTFIX
7 years ago
7 years ago

People

(Reporter: rnewman, Assigned: rnewman)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(3 attachments)

(Assignee)

Description

7 years ago
I noticed some code duplication between my favicons sync work and our repositories/async rewrite.

Here's a little chunk of code to make asynchronous querying easier.
(Assignee)

Comment 1

7 years ago
Created attachment 565111 [details] [diff] [review]
Part 1: Add AsyncQueryCallback, Async.queryForRows helper. v1
Attachment #565111 - Flags: review?(philipp)
(Assignee)

Comment 2

7 years ago
Created attachment 565112 [details] [diff] [review]
Part 2: Tests for Async.queryForRows. v1

Tests are fairly sparse, because of Part 3…
Attachment #565112 - Flags: review?(philipp)
(Assignee)

Comment 3

7 years ago
Created attachment 565113 [details] [diff] [review]
Part 3: Rewrite querySpinningly in terms of queryForRows. v1
Attachment #565113 - Flags: review?(philipp)
Comment on attachment 565111 [details] [diff] [review]
Part 1: Add AsyncQueryCallback, Async.queryForRows helper. v1

>+/**
>+ * Create a mozIStorageStatementCallback object for use in asynchronous
>+ * queries.
>+ *
>+ * Usage:
>+ *
>+ *   function getGUID(row) {
>+ *     return row.guid;
>+ *   }
>+ *   let stmt = helper.getStatement("SELECT guid ...");
>+ *   let cb = new QueryCallback(stmt, getGUID, function (err, guids) {
>+ *     _("Got GUIDs! " + JSON.stringify(guids[0]));
>+ *   };
>+ *   cb.run();

Summarizing IRL discussion:

This is a bit more generalization than we need for the async stuff, at the expense of lots of function calls and potential inner closures. What do we need for the async APIs? We need:

(a) a way to get an array of just one column's values (the GUIDs), e.g.:

  guidsSince: function guidsSince(timestamp, guidsCallback) {
    queryForRows(this.guidsStmt, "guid", guidsCallback);
  }

(b) a way to get rows and create objects from them, passed to a fetchCallback. This one has a particular signature, expecting the DONE sentinel value at handleCompletion. Your transform function would work well here:

  fetchSince: function fetchSince(timestamp, fetchCallback) {
    let stmt = this.recordsSinceStmt;
    stmt.timestamp = timestamp;
    new FetchQueryCallback(stmt, this.dbRowToRecord, fetchCallback);
  },

  dbRowToRecord: function dbRowToRecord(row) {
    return {id:    row.getResultByName("guids"),
            title: row.getResultByName("title"),
            ttl:   GENERIC_COLLECTION_TTL,
            etc.};
  },

As for the favicon use case, it seems you only ever need 1 result anyway, which means you can optimize the mozIStorageCallback implementation for that. Yes, this would mean we would have a bunch of different mozIStorageCallback implementations in our code, but they're all serving a slight different purpose.
Attachment #565111 - Flags: review?(philipp) → review-
(Assignee)

Comment 5

7 years ago
Will work in `repositories`, and handle the favicons case separately.
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → WONTFIX
Attachment #565112 - Flags: review?(philipp)
Attachment #565113 - Flags: review?(philipp)
You need to log in before you can comment on or make changes to this bug.