stats DB for monitor #99

antonis.lempesis · 2021-03-03T00:26:03+01:00

antonis.lempesis commented

2021-03-03 00:26:03 +01:00

creating monitor db
correctly computing stats
new parameters:
monitor_db_name=openaire_prod_stats_monitor_{date}
monitor_db_shadow_name=openaire_prod_stats_monitor_shadow

1. creating monitor db 2. correctly computing stats 3. new parameters: monitor_db_name=openaire_prod_stats_monitor_{date} monitor_db_shadow_name=openaire_prod_stats_monitor_shadow

claudio.atzori changed title from ~~master~~ to stats DB for monitor

2021-03-03 09:23:13 +01:00

claudio.atzori self-assigned this 2021-03-03 09:59:41 +01:00

claudio.atzori added the

enhancement

label 2021-03-03 09:59:44 +01:00

claudio.atzori referenced this issue from a commit

2021-03-03 10:29:09 +01:00

Merge pull request 'stats DB for monitor' (#99) from antonis.lempesis/dnet-hadoop:master into master Looks good to me, just a note on the parsing of the citations: since the last version, IIS produces citations as proper relationships among results. This is what we got already in the BETA graph ``` count r.reltype r.subreltype r.relclass 62.129.254 resultResult citation cites 62.043.309 resultResult citation isCitedBy ``` Thus, I suggest to move away from the current property based implementation for the extraction of the citation links and start relying on the relationships instead.

claudio.atzori closed this pull request

2021-03-03 10:29:09 +01:00

claudio.atzori commented

2021-03-03 10:29:53 +01:00

Looks good to me, just a note on the parsing of the citations: since the last version, IIS produces citations as proper relationships among results. This is what we got already in the BETA graph

count		r.reltype	r.subreltype	r.relclass
62.129.254	resultResult	citation	cites
62.043.309	resultResult	citation	isCitedBy

Thus, I suggest to move away from the current property based implementation for the extraction of the citation links and start relying on the relationships instead.

Looks good to me, just a note on the parsing of the citations: since the last version, IIS produces citations as proper relationships among results. This is what we got already in the BETA graph ``` count r.reltype r.subreltype r.relclass 62.129.254 resultResult citation cites 62.043.309 resultResult citation isCitedBy ``` Thus, I suggest to move away from the current property based implementation for the extraction of the citation links and start relying on the relationships instead.

Sign in to join this conversation.

No reviewers