-
Notifications
You must be signed in to change notification settings - Fork 46
Conversation
@ryandeivert This is awesome, thank you! Been on my backlog for a bit, so I definitely appreciate the contribution. Will take a look at this within the next week. |
Any updates on this? |
Hi @juanqui - I unfortunately don't work at AWS anymore so not able to merge this in. I'll try to ping somebody on my old team there, but no guarantees. Feel free to fork this, though! |
I did a manual zip deploy of this and ran against S3 access logs using Glue 2.0 Python3 with |
Any news on this PR? Why it has not been merged in? |
@serverlesspolska I was the primary maintainer of this repo, but then left Amazon and wasn't able to find a replacement. That said, I'm back at Amazon now so will take a look at what needs to be done to update this / if we should continue maintaining it. A lot has changed in the past year. :) |
Due to python3 changes, this script is incompatible with python2 now so making a major version bump.
Thanks @ryandeivert for submitting this PR and I apologize for taking checks notes almost 2 years to merge it in. Keep an eye on the repo as there have been a lot of changes in Glue since then and Blueprints/Workflows may be the path forward. |
Description of changes:
AWS Glue recently launched support for python3
Including updates to convert this project to python3. The changes include the following:
Testing:
I ensured all tests are still passing using
make test