botify-labs / simpleflow

Python library for dataflow programming.
https://botify-labs.github.com/simpleflow/
MIT License
68 stars 24 forks source link

Upgrade to boto3 #419

Closed jbbarth closed 11 months ago

jbbarth commented 1 year ago

Hey @ybastide 👋 I hope everything's good on your side

I'd be VERY interested to upgrade simpleflow to use boto3:

Let me know what you think

ybastide commented 1 year ago

👋 Hi! Long time no see 🙂

I'd be delighted, obviously 🎉 Do you plan to do it?

jbbarth commented 1 year ago

Yes I'm happy to do it, I'll take a look in the next weeks and submit a PR. What I'm not 100% clear about is the potential boto.cfg and AWS_* env vars on your infra, but we can deal with that later.

ybastide commented 1 year ago

We're using both, so it will inevitably be OK 🙃

jbbarth commented 1 year ago

@ybastide I started looking into it but things are complicated on the "swf" module querysets:

I'm wondering:

I started with "1 = yes, 2 = B", but I'm curious what's your take on this. I'd be fine with "1 = no" or "1 = yes, 2 = C" 😅

jbbarth commented 1 year ago

Nevermind, it's not that hard to go with "1 = yes, 2 = B" in the end. PRs soon.

jbbarth commented 11 months ago

423 is merged, yoohoo!