Fiche sur les bases de données - [merged] #136

Closed linogaliana closed 3 years ago

In GitLab by @linogaliana on Apr 11, 2020, 08:34

_Merges fichesgbd -> master

J'ouvre la MR pour faciliter le processus de review (promis moins de spam que sur !27 )

Close #37

In GitLab by @pierre-lamarche on Apr 19, 2020, 09:59

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 107

changed this line in version 23 of the diff

In GitLab by @pierre-lamarche on Apr 19, 2020, 09:59

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 176

changed this line in version 23 of the diff

In GitLab by @pierre-lamarche on Apr 19, 2020, 09:59

added 1 commit

81acb80c - Réorganisation de la fiche conformément à la proposition d'Olivier

Compare with previous version

In GitLab by @oliviermeslin on Apr 19, 2020, 11:58

added 6 commits

95cf6d05 - Formulation
7d2b1ebb - Complément sur SQL
8aa7dbb5 - Ajout d'une petite section sur le workflow
546603b4 - Reformulation recommandation
23d2b845 - Début de réorganisation du plan
5356ead1 - Solve conflict

Compare with previous version

In GitLab by @oliviermeslin on Apr 19, 2020, 14:14

added 2 commits

26774b90 - Rédaction partie workflow avec les BDD
869f7ea1 - Réorganisation du plan

Compare with previous version

In GitLab by @oliviermeslin on Apr 19, 2020, 14:20

added 1 commit

c6f846bf - Petites reformulations

Compare with previous version

In GitLab by @oliviermeslin on Apr 19, 2020, 14:27

added 1 commit

fe2a1477 - Amélioration de la description du workflow

Compare with previous version

In GitLab by @oliviermeslin on Apr 19, 2020, 14:37

added 1 commit

9c89678f - Des détails sur les exemples de requête Fideli

Compare with previous version

In GitLab by @oliviermeslin on Apr 19, 2020, 14:55

added 1 commit

273d8150 - Pas mal de reformulations

Compare with previous version

In GitLab by @oliviermeslin on Apr 19, 2020, 14:59

added 1 commit

05bb56e7 - Reformulation et suppression de doublon

Compare with previous version

In GitLab by @pierre-lamarche on Apr 19, 2020, 15:34

added 2 commits

bd1ca552 - Reformulation partie workflow back-end/front-end
45692bae - Reformulation répertoire => schéma pour mieux coller avec la terminologie BDD

Compare with previous version

In GitLab by @pierre-lamarche on Apr 19, 2020, 16:48

added 1 commit

53e329b0 - Correction spécification de schéma sur dbplyr (shame on me)

Compare with previous version

In GitLab by @oliviermeslin on Apr 20, 2020, 05:58

added 2 commits

b6e38031 - Les points virgules c'est ma passion
7a88e597 - Alléger les recommandations

Compare with previous version

In GitLab by @oliviermeslin on Apr 20, 2020, 06:09

approved this merge request

In GitLab by @oliviermeslin on Apr 20, 2020, 06:23

added 2 commits

20e134cc - Ajouts des conseils de Papy Insee
c3f825e9 - Ajout de dbplyr dans les recommandations

Compare with previous version

In GitLab by @oliviermeslin on Apr 20, 2020, 06:23

approved this merge request

In GitLab by @oliviermeslin on Apr 20, 2020, 06:24

@linogaliana, @RLesur : que pensez-vous de la fiche en l'état?

Pierre et moi pensons que nous ne sommes pas loin de pouvoir la merger.

In GitLab by @linogaliana on Apr 20, 2020, 07:25

Je valide ! :thumbsup:

In GitLab by @linogaliana on Apr 20, 2020, 07:26

resolved all threads

In GitLab by @linogaliana on Apr 20, 2020, 07:27

unmarked as a Work In Progress

In GitLab by @linogaliana on Apr 20, 2020, 07:27

changed the description

In GitLab by @linogaliana on Apr 20, 2020, 07:29

@oliviermeslin comme j'ai créé la MR je ne peux pas l'approuver mais @pierre-lamarche (ou @RLesur ) doit pouvoir le faire pour atteindre le quorum nécessaire

In GitLab by @pierre-lamarche on Apr 20, 2020, 07:29

approved this merge request

In GitLab by @gillesfidani on Apr 20, 2020, 08:50

added 3 commits

09e18d13 - puisse devient peut
c7ec4c72 - ajout d'un g à Postgres
e2ecc664 - connection devient connexion

Compare with previous version

In GitLab by @oliviermeslin on Apr 20, 2020, 08:50

approved this merge request

In GitLab by @pierre-lamarche on Apr 20, 2020, 08:56

approved this merge request

In GitLab by @gillesfidani on Apr 20, 2020, 08:58

approved this merge request

In GitLab by @oliviermeslin on Apr 20, 2020, 08:59

mentioned in commit 917b8387b36d7a645e23e91bcb4e8cd6e901f306

In GitLab by @oliviermeslin on Apr 20, 2020, 08:59

merged

In GitLab by @RLesur on Apr 20, 2020, 13:56

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 112

Dites, êtes-vous certains que les arguments de la méthode dbConnect() pour les objets de classe OdbcDriver prennent bien des initiales en majuscule ?
Je n'utilise pas le package odbc, mais si je comprends bien, l'objectif de mettre des majuscules est de passer ces arguments au dots argument pour qu'ils soient rajoutés à la chaîne de connexion, c'est bien ça ?

In GitLab by @oliviermeslin on Apr 20, 2020, 14:43

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 112

Alors j'ai réussi à me connecter à une base de données PostgreSQL avec le code suivant:

conn <- DBI::dbConnect(
  odbc::odbc(),
  driver = "PostgreSQL Unicode(x64)",
  server = "bspocimmoldb01.ad.insee.intra",
  database = "ri_pg_pocimmo_bs01",
  port = 1983,
  uid =  rstudioapi::askForPassword("Nom d'utilisateur"),
  pwd = rstudioapi::askForPassword("Mot de passe"))

Ca marche aussi avec le nom des arguments en majuscules. Est-ce que ça répond à ta question?

In GitLab by @RLesur on Apr 20, 2020, 15:36

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 112

Oui, en grande partie. En fait, ce qui m'avait intrigué, c'était en lisant la page d'aide suivante :

?`dbConnect,OdbcDriver-method`

dans laquelle on voit que tous les arguments sont en minuscules.

Je pense que même si ça donne le même résultat, la mécanique n'est pas tout à fait la même. Dans le cas où les arguments sont en majuscule, ils sont tous passés à l'argument ... alors qu'avec les minuscules, ce sont les arguments de la méthode dbConnect() pour l'objet de classe OdbcDriver qui sont renseignés.

Si les deux fonctionnent, tant mieux !

Cependant, en tant que lecteur, j'aurais une petite préférence pour l'utilisation des arguments en minuscule. Je vous laisse voir et décider !

In GitLab by @oliviermeslin on Apr 20, 2020, 15:57

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 112

je suis d'accord, je le fais

In GitLab by @oliviermeslin on Apr 20, 2020, 16:11

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 112

Fait avec bade2864

In GitLab by @oliviermeslin on Apr 20, 2020, 16:11

resolved all threads

In GitLab by @pierre-lamarche on Apr 20, 2020, 19:10

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 112

En effet, dans ce cas, c'est mieux avec les minuscules. Je m'étais calé sur la documentation de RStudio : https://db.rstudio.com/databases/postgresql/#connection-settings sans en interroger la qualité. Merci pour la précision.

In GitLab by @RLesur on Apr 21, 2020, 07:30

Commented on 03_Fiches_thematiques/Fiche_connexion_bdd.Rmd line 112

Je pense que c'est parce que cette doc n'a pas été mise à jour. En effet, il fut un temps où il fallait utiliser les majuscules : https://github.com/r-dbi/odbc/blob/8c0d3ec61ad7b46d8c6c05cfcaacd3719b58e70b/R/Connection.R.

Mais depuis https://github.com/r-dbi/odbc/commit/728aebae136965abbf82bd73ea40a8132ad48b4c#diff-e1e1d3d40573127e9ee0480caf1283d6, les arguments ont été ajoutés.

InseeFrLab / utilitR

Fiche sur les bases de données - [merged] #136