Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Udemy #610

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
Open

Udemy #610

wants to merge 19 commits into from

Conversation

sabinem
Copy link

@sabinem sabinem commented Dec 4, 2020

No description provided.

Maennel added 19 commits December 2, 2020 10:42
change date in tuto.py dag to current date
44c0512
add redis to the docker setup
2025edf
add assignment1
16f03e3
start of sales project: add mysql to docker setup
f974a2e
add and mount mysql configuration file
cb5b6df
add store file data
00397ea
add smtp to docker set up
c6a055e
add 1. task to dag: check that file exists
3800292
add second task to dag: python operatot to clean the csv file
adc9797
add 3. task: create mysql table
028fa4d
add task 4: insert into mysql table
12b7427
add task 5: select from table to get aggregations
0218b87
add task 7 and 8: copy files
bcbecbc
add output files to gitignore
cd7172f
document mysql handeling
313acab
document docker commands
27949b2
reformat create table sql command
a705253
add task 9 to dag: rename input file by adding a timestamp
d1d7c8b
finished store_dag before using context
809416f
shihabcsedu09
shihabcsedu09 suggested changes Dec 30, 2020
View reviewed changes
Copy link

@shihabcsedu09 shihabcsedu09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sabinem This should not be part of this repo.

@puckel

shihabcsedu09
shihabcsedu09 suggested changes Jan 19, 2021
View reviewed changes
Copy link

@shihabcsedu09 shihabcsedu09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not relevant

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shihabcsedu09 shihabcsedu09 shihabcsedu09 requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sabinem @shihabcsedu09