Code RoomException type lost in wrapper
MediumPrep Room Coding #2120

Exception type lost in wrapper

Code reviewDistributed systemsCode quality & reviewMid–Senior~16 min

Review this Python data-loader. The caller needs to retry on a transient `ConnectionError` but give up on a `SchemaError`.

What a strong answer looks like

Separate real bugs from style. Rank issues by severity, point at the root cause rather than the symptom, and suggest a concrete fix, specific and kind.

0:00 of about 16 min
Mark a line and say what kind of problem it is.0 findings
1def load_partition(part):
2 try:
3 return store.read(part)
4 except Exception as e:
5 raise RuntimeError(f"failed to load {part}: {e}")
6 
7def run(parts):
8 for p in parts:
9 try:
10 load_partition(p)
11 except ConnectionError:
12 retry_later(p)
13 except SchemaError:
14 quarantine(p)
Which questions mattered is sealed until you submit. Telling you now would just be handing over the edge cases.