This repository has been archived by the owner on Aug 24, 2022. It is now read-only.
Extract committer from commit data #59
Comments
|
Hi Claus,
Am 22/06/2017 um 17:16 schrieb Claus Hunsen:
Right now, only the commit date is stored in the |commit| table of the
DB, but it would be also a good idea to get the committer, too (as a
foreign key to the |person| table, of course).
Are there any obstacles to get the committer ID, too?
none at all -- in case you did not do implement the change yet, let's
briefly discuss this because I have a few other changes to the DB model
in the pipeline and could quickly do this myself (which is I easier
than integrating two DB changes, which always turns out to be quite
a hassle).
In particular, I will be adding exact size information for the files
that are touched by commits because this information is right now
hard to compute.
Best regards, Wolfgang
…
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#59>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABbE1wlsFR18AVmv3HlsIgG_taAC_rihks5sGoU8gaJpZM4OCciV>.
|
|
I have not implemented anything yet. I would be glad if you implement this along with your changes. 😉 |
|
Any news on this, @wolfgangmauerer? |
clhunsen
added a commit
to se-sic/codeface
that referenced
this issue
Dec 15, 2017
For better traceability and to enable subsequent analyses, we now always
store the committer of a commit in the database during commit analysis
in addition to the commit's author.
To add the missing column in existing databases created with previous
schemas, run the following command inside MySQL:
ALTER TABLE `codeface`.`commit`
ADD COLUMN `committer`
BIGINT NOT NULL COMMENT ''
AFTER `author`,
ADD CONSTRAINT `commit_committer`
FOREIGN KEY (`committer`)
REFERENCES `codeface`.`person` (`id`)
ON DELETE CASCADE
ON UPDATE CASCADE,
DROP INDEX `commit_person_idx`,
ADD INDEX `commit_person_idx`
(`author` ASC, `committer` ASC) COMMENT '';
This fixes siemens#59.
Signed-off-by: Claus Hunsen <[email protected]>
clhunsen
added a commit
to se-sic/codeface
that referenced
this issue
Jan 16, 2018
For better traceability and to enable subsequent analyses, we now always
store the committer of a commit in the database during commit analysis
in addition to the commit's author.
To add the missing column in existing databases created with previous
schemas, run the following command inside MySQL:
ALTER TABLE `codeface`.`commit`
ADD COLUMN `committer`
BIGINT NOT NULL COMMENT ''
AFTER `author`,
ADD CONSTRAINT `commit_committer`
FOREIGN KEY (`committer`)
REFERENCES `codeface`.`person` (`id`)
ON DELETE CASCADE
ON UPDATE CASCADE,
DROP INDEX `commit_person_idx`,
ADD INDEX `commit_person_idx`
(`author` ASC, `committer` ASC) COMMENT '';
This fixes siemens#59.
Signed-off-by: Claus Hunsen <[email protected]>
ecklbarb
pushed a commit
to ecklbarb/codeface
that referenced
this issue
Feb 7, 2018
For better traceability and to enable subsequent analyses, we now always
store the committer of a commit in the database during commit analysis
in addition to the commit's author.
To add the missing column in existing databases created with previous
schemas, run the following command inside MySQL:
ALTER TABLE `codeface`.`commit`
ADD COLUMN `committer`
BIGINT NOT NULL COMMENT ''
AFTER `author`,
ADD CONSTRAINT `commit_committer`
FOREIGN KEY (`committer`)
REFERENCES `codeface`.`person` (`id`)
ON DELETE CASCADE
ON UPDATE CASCADE,
DROP INDEX `commit_person_idx`,
ADD INDEX `commit_person_idx`
(`author` ASC, `committer` ASC) COMMENT '';
This fixes siemens#59.
Signed-off-by: Claus Hunsen <[email protected]>
clhunsen
added a commit
to se-sic/codeface
that referenced
this issue
Mar 22, 2018
For better traceability and to enable subsequent analyses, we now always
store the committer of a commit in the database during commit analysis
in addition to the commit's author.
To add the missing column in existing databases created with previous
schemas, run the following command inside MySQL:
ALTER TABLE `codeface`.`commit`
ADD COLUMN `committer`
BIGINT NOT NULL COMMENT ''
AFTER `author`,
ADD CONSTRAINT `commit_committer`
FOREIGN KEY (`committer`)
REFERENCES `codeface`.`person` (`id`)
ON DELETE CASCADE
ON UPDATE CASCADE,
DROP INDEX `commit_person_idx`,
ADD INDEX `commit_person_idx`
(`author` ASC, `committer` ASC) COMMENT '';
This fixes siemens#59.
Signed-off-by: Claus Hunsen <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Right now, only the commit date is stored in the
committable of the DB, but it would be also a good idea to get the committer, too (as a foreign key to thepersontable, of course).Are there any obstacles to get the committer ID, too?
The text was updated successfully, but these errors were encountered: