manually clear key when Transient Errors encounted?

[graphql]

manually clear key when Transient Errors encounted?

[ThisIsMissEm]

It'd seem so; I think the docs probably need to be updated.

[leebyron]

I just made some updates to Readme that hopefully improve this. Your suggestion is half-correct. If the entire batch request fails, then cache is cleared. If individual values are provided Error, those are cached. I updated the language to make this clear.

[luckydrq]

Thanks, i got your point.

However, in real world from what i see, it rarely happens when individual values are provided Error, because batch function is designed to reduce network cost of fetching data to backend. For example, a batch function to load data from a rational database may look like this:

var batchFn = ids => {
  return new Promise((resolve, reject) => {
    sqlRun(`SELECT * FROM user_table WHERE ID IN (${ids.join(',')})`, (err, records) => {
      if (err) {
        return reject(err);
      } else {
        return resolve(records);
      }
    });  
  });
};

rather than

var batchFn = ids => {
  return Promise.all(ids.map(id => {
    return new Promise((resolve, reject) => {
      sqlRun(`SELECT * FROM user_table WHERE id=${id}`, (err, record) => {
        if (err) {
          return reject(err);
        } else {
          return resolve(record);
        }
      });
    });
  }));
}

[leebyron]

It certainly depends on the system you're communicating with, and of course it's up to how you implement your batching function. There is of course the case where the batch itself fails, which is well illustrated by your first example. There is also a case where your batch function may return a combination of values and errors. That is - the batch load itself succeeded however some of the individual operations within the batch failed.

[luckydrq]

Agreed, just to share my thoughts, thanks!