search

chaoss / grimoirelab-graal

A Generic Repository AnALyzer
GNU General Public License v3.0
21 stars 62 forks source link

[graal] Checkout log an issue in case of large repositories #33

Closed inishchith closed 5 years ago

inishchith commented 5 years ago

Issue Scenario: I found this out when I was working on the integration task of Graal with ELK (https://github.com/inishchith/gsoc-graal/issues/4). which causes the checkout log (for the number of commits):

Git repository <repo-path> checked out!

This wouldn't be a good to have in case of repositories of a large number of commits causing a console log for eg. [image attached below on the first run]

Screenshot 2019-06-04 at 6 45 07 PM

@valeriocos @jgbarah I was thinking if we could remove the logger for the checkout purpose (a small change but would lead to better log). Would like to have your thoughts :)

Thanks!

valeriocos commented 5 years ago

Thank you for reporting this potential issue @inishchith . Maybe we could change the log level to debug (this could be applied to most messages except the one in graal.fetch_items. What do you think ?

inishchith commented 5 years ago

@valeriocos Thanks for the quick response! Yes. That would be better indeed. I'd propose a fix for this in some time :)

valeriocos commented 5 years ago

great! thanks @inishchith