Summary

The use of WT_IGNORE_RET ignores the error returned by __wt_buf_set. We should fix this.

    if (arg_count > 1) {
        datastore_key = va_arg(ap, WT_ITEM *);
        WT_IGNORE_RET(__wt_buf_set(
          session, hs_cursor->datastore_key, datastore_key->data, datastore_key->size));
        F_SET(hs_cursor, WT_HS_CUR_KEY_SET);
    } else {

Motivation

Does this affect any team outside of WT?

No

How likely is it that this use case or problem will occur?

N/A

If the problem does occur, what are the consequences and how severe are they?

N/A

Is this issue urgent?

Nothing strict, but more testing is more good.

Acceptance Criteria (Definition of Done)

Handle the error in a reasonable way.

Testing

N/A

Documentation update

N/A