Fixed example code

[dividuum]

Python code works and matches the documentation.

[alecgrieser]

Looking at your comment in #217, yeah, I think that that is the right way to go. (It would at least make Java and Python consistent). You could also (instead of collecting the iterator as a list) use a for-else construct or whatever the most pythonic way to grab the first item from an iterator is (I'm not sure).

[dividuum]

Right. I played a bit more with foundationdb and the following also works as an empty range just falls through and returns None.

@fdb.transactional
def get_user(tr, ID):
    for key, value in tr.get_range_startswith(user[ID], limit=1):
        return value

[alecgrieser]

Hm, I think I'd prefer if it explicitly returned None on the failure case (even though it doesn't have to, strictly speaking), but other that, looks good.

[alexmiller-apple]

@dividuum, any chance you have some time to address the review feedback so that we can merge the PR? 🙂

[alexmiller-apple]

Hi @dividuum,

We're wrapping up the work that was slated for the 6.1 release, and will start looking to cut the release branch. Any chance you'd have the time to revisit this PR? If this were to be merged in the next week or two, then it would be included in the 6.1 release. Otherwise, it'd likely be ~6mo until the next release.

Thanks,
Alex

[ajbeamon]

Resolved in #1218