Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Udemy #610

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
Open
sabinem wants to merge 19 commits into
base: master
Choose a base branch
Loading
from
Open

Udemy #610

sabinem 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

1 more reviewer

@shihabcsedu09 shihabcsedu09 shihabcsedu09 requested changes

Reviewers whose approvals may not affect merge requirements

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
Morty Proxy This is a proxified and sanitized view of the page, visit original site.