Skip to content
Snippets Groups Projects

WIP: Create scripts to update db with valid data

Closed Raul Almeida requested to merge updateMachineUse into development

The scripts export data from varian db to pinsis db keeping only truly executed treatment sessions. The table machineuse_upd is used as a temporary data holder and machineuse is appended with the new registries. Data from dbtest.machineuse should be anonimized before these scripts are ran.

Edited by Alessandro Elias

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • @rgpa18

    Rejeitado! Motivo:

    Não redirecione nada para /dev/null ao menos que gere uma flag [--quiet|-q]. ainda não é necessário as flags, pois deve ser um script simples, apenas deixe ir stdout para tela.

    copyCSV.sh

    Separe os comando sql. coloque em array as tabelas e expanda no for.

    exportCSV.sh

    Separe o sql. coloque em um array as tabelas e expanda no for.

    adjustDataBase.sh

    veja que você dispara todos os scripts a partir deste, logo todas as váriaveis são dependentes. Faça checagem de todas as variáveis de database_info.sh; include host para a conexão do postgres.

    filterMachineUse.sh

    falta flag -h no comando psql, no caso da pinsisdev funciona, pq esta configurado como trust localhost, caso execute o script em outro ambiente não funcionará.

    Apesar de estar funcional a sua lógica, não é muito legível.

    Separe o comando sql em uma variável, assim ficará mais legível, e.g.: psql -h ${host} -U ${user_psql} -d

    {db_psql} -c "
    SUA_VAR"

    não abuse do shell, quanto mais legível melhor, o c3 tem muita rotatividade de bolsistas.

    updateMachineUse.sh

    Separe sql.

  • Alessandro Elias marked as a Work In Progress

    marked as a Work In Progress

  • Alessandro Elias changed the description

    changed the description

Please register or sign in to reply
Loading