Avoid KeyError when filtering fetchable partitions

[dpkp]

I noticed this failure in a failed travis run. If we attempt to remove a partition that is not currently fetchable, the set will raise a KeyError. To avoid, we test that the element is in the set before removing it.

[tvoinarovskyi]

jfyi, discard can be used here, as it's a set.

>>> x = set([1, 2, 3])
>>> x.remove(0)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 0
>>> x.discard(0)
>>> x
{1, 2, 3}
>>> x.discard(1)
>>> x
{2, 3}

[dpkp]

TIL! On Jan 11, 2018 6:49 PM, "Taras Voinarovskyi" <notifications@github.com> wrote: jfyi, discard can be used here, as it's a set.

>> x = set([1, 2, 3])>>> x.remove(0)

Traceback (most recent call last): File "<stdin>", line 1, in <module>KeyError: 0>>> x.discard(0)>>> x {1, 2, 3}>>> x.discard(1)>>> x {2, 3} — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1344 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAzetGpFfefUmNtbjptQj9yEqWvkbE1Wks5tJshWgaJpZM4Rbs_x> .