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

Improve date storage #9

Open
slingerbv opened this issue Jun 27, 2023 · 2 comments
Open

Improve date storage #9

slingerbv opened this issue Jun 27, 2023 · 2 comments

Comments

@slingerbv
Copy link
Contributor

Currently, we're not storing the date of a method's last change. We want to have better dating for each method.

@slingerbv
Copy link
Contributor Author

@ra5put1n please elaborate

@ra5put1n
Copy link
Collaborator

ra5put1n commented Sep 6, 2023

Currently, the miner stores the date of method as the date of the last edit for the file. This is not accurate as the method could be added much before the last file edit.

To solve this, we can either recursively check the file changes in every commit for every file during the parsing process. That would be resource intensive and will increase the take the time taken for parsing a repository (impact needs experimental verification).

A better approach would be to have a service which does the date computation when a check request is recieved. That is, whenever a project is queried, all the matched functions' dates are fetched through the file commit history. This way, we only perform computation for matched functions while also not consuming time while mining.

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

No branches or pull requests

2 participants