Closed tatiana closed 1 year ago
Over all LGTM.
I have few open questions though:
- Do you think we should add the cross database transfers example as well in read the docs?
- Do you think we should this example as well: f3caa09 I feel we could introduce example DAG section and add it there.
I'm happy to add those, please, feel free to suggest where you think they'd be better placed!
@sunank200 @utkarsharma2 thanks for the reviews - I tried to address the feedback.
In the case of the example DAGs, I added a link to our example DAGs directory. I also tried to make it explicit what types of transfer we currently support (object store -> database, database -> object store, database -> database).
Please, let me know your thoughts
There are still a couple of improvements that we could make, which can be addressed in a separate PR: