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.

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

feat: add platform to events table #1438

Merged
merged 1 commit into from
Dec 11, 2023

Conversation

frank-zsy
Copy link
Contributor

close #1437

@frank-zsy frank-zsy marked this pull request as draft December 2, 2023 08:51
@frank-zsy
Copy link
Contributor Author

frank-zsy commented Dec 3, 2023

As there are lots of changes made in this PR, I will record the changes while making them.

  • Add platform column in the database.
  • Reimport all the GitHub and Gitee events into events table and delete gh_events table
  • Modify the database schema file
  • Modify label data structure. Put repo/org/user names from comments to name field to make sure the data exists after parsing to JSON
  • Modify github_repo to github.repo and same to orgs and users. So we can easily add other platforms data.
  • Modify the label_data_utils to fit the new label structure.
  • Modify the query option structure.
  • Modify the underlying libs to fit the new query option structure.

@frank-zsy frank-zsy force-pushed the feat/add-platform branch 7 times, most recently from d9d9ed0 to 1a80224 Compare December 8, 2023 12:02
@frank-zsy
Copy link
Contributor Author

I will merge this PR first since all basic functionalities are already ready to use, more detailed changed will be added in the future.

@frank-zsy frank-zsy marked this pull request as ready for review December 11, 2023 14:25
@frank-zsy frank-zsy merged commit 9ca4fbb into X-lab2017:master Dec 11, 2023
1 check passed
@frank-zsy frank-zsy deleted the feat/add-platform branch December 11, 2023 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Add platform column to ClickHouse database.
1 participant