COOL-cohort / COOL

the source code of the COOL system
https://www.comp.nus.edu.sg/~dbsystem/cool/
Apache License 2.0
45 stars 16 forks source link

Add ecommerce SQL analysis, update query logic, add comments #54

Closed NLGithubWP closed 2 years ago

NLGithubWP commented 2 years ago
  1. Move all data into the dataset folder, sort e-commerce datasets on data
  2. Add SQL analysis information
  3. Add test function to test e-commerce - query
  4. Add comments to query/ writer
  5. Remove Unused variables
  6. Fix action groupBy bug, GroupBy action time with NUMERIC, (Fix issue https://github.com/COOL-cohort/COOL/issues/55)
  7. Enable GroupBy action time on month
  8. We run python scripts to count the distinct product_id under each time_range, the result is not the same as Product_Retention.csv,
qlinsey commented 2 years ago

Is this to me?

Get Outlook for iOShttps://aka.ms/o0ukef


From: working @.> Sent: Wednesday, July 13, 2022 1:57:00 AM To: COOL-cohort/COOL @.> Cc: qlinsey @.>; Mention @.> Subject: Re: [COOL-cohort/COOL] Add ecommerce SQL analysis, update query logic, add comments (PR #54)

@NLGithubWP commented on this pull request.


In datasets/ecommerce/table.yamlhttps://github.com/COOL-cohort/COOL/pull/54#discussion_r919826770:

@@ -0,0 +1,29 @@ +charset: utf-8

@qlinseyhttps://github.com/qlinsey this is the table YAML, update-sample.csv is the datasets

— Reply to this email directly, view it on GitHubhttps://github.com/COOL-cohort/COOL/pull/54#pullrequestreview-1037008924, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABI2ZIEAFLVJIRZSCUIVG3TVT2AFZANCNFSM5YR64ARA. You are receiving this because you were mentioned.Message ID: @.***>

NLGithubWP commented 2 years ago

Is this to me? Get Outlook for iOShttps://aka.ms/o0ukef ____ From: working @.> Sent: Wednesday, July 13, 2022 1:57:00 AM To: COOL-cohort/COOL @.> Cc: qlinsey @.>; Mention @.> Subject: Re: [COOL-cohort/COOL] Add ecommerce SQL analysis, update query logic, add comments (PR #54) @NLGithubWP commented on this pull request. ____ In datasets/ecommerce/table.yaml<#54 (comment)>: @@ -0,0 +1,29 @@ +charset: utf-8 @qlinseyhttps://github.com/qlinsey this is the table YAML, update-sample.csv is the datasets — Reply to this email directly, view it on GitHub<#54 (review)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABI2ZIEAFLVJIRZSCUIVG3TVT2AFZANCNFSM5YR64ARA. You are receiving this because you were mentioned.Message ID: @.***>

Sorry, it's not to you, I misclicked.