Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat identifier cols #39

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

Feat identifier cols #39

wants to merge 23 commits into from

Conversation

kapsner
Copy link
Member

@kapsner kapsner commented Mar 22, 2022

addresses #31

@makampf makampf force-pushed the feat_identifier_cols branch 2 times, most recently from 1841d1f to c1f09dc Compare August 19, 2022 10:08
@makampf
Copy link
Member

makampf commented Aug 19, 2022

Hallo @kapsner, es gibt beim Build einen Fehler im Test. Beim Initialisieren wird die schema.sql ausgeführt, aber diese kennt kein JSONB. Gibt es hier einen Konflikt zwischen verschiedenen SQL-Dialekten?

Fehler: https://github.com/miracum/fhir-gateway/runs/7916249699?check_suite_focus=true#step:10:566

Mir kam es eh schon eigenartig vor: Der SQLFLUFF Linter hat dauernd Fehler geschmissen, egal welchen Dialekt ich konfiguriert habe. Scheinbar ist es eine Mischung aus Postgres und MSSQL?
Oder hast du eine Idee, woran es liegt?

@makampf makampf force-pushed the feat_identifier_cols branch 3 times, most recently from b38b479 to db845be Compare August 19, 2022 17:49
renovate bot and others added 16 commits August 19, 2022 19:55
@renovate @makampf
fix(deps): update dependency ca.uhn.hapi.fhir:hapi-fhir-base to v6
c9b3ad3
@renovate-bot @makampf
chore(deps): update docker/build-push-action action to v3
9c852d8
@renovate @makampf
chore(deps): update docker.vectorized.io/vectorized/redpanda docker t…
2b0dd18 
…ag to v22
@renovate @makampf
chore(deps): update actions/checkout action to v3
e3cee46
@renovate-bot @makampf
chore(deps): update actions/cache action to v3
91d4d93
@renovate @makampf
chore(deps): update actions/setup-node action to v3
a821897
@renovate @makampf
chore(deps): update actions/setup-python action to v4
c72305c
@renovate @makampf
chore(deps): update all non-major dependencies docker tags
7d926e3
@renovate @makampf
chore(deps): update all non-major dependencies
46b8d33
@kapsner @makampf
feat: partitioning by default
873d315
@kapsner @makampf
chore: updated readme to current schema creation
189c936
@kapsner @makampf
feat: new id columns for patient id and encounter id
513fa33
@kapsner @makampf
chore: updated todos for miracum#42, miracum#31
4c2772e
@makampf
chore(ci): disable sqlfluff in ci bc not working
0363f6e
@makampf
chore(ci): apply spotless to fix ci
2d1bb09
@makampf
chore(ci): bump codeql action
e42f63c
@makampf makampf removed their assignment Oct 13, 2022
@makampf
Copy link
Member

makampf commented Oct 13, 2022

Moin @kapsner, können wir hier nochmal den Bedarf/die Relevanz absprechen? Wenn das nicht mehr benötigt wird, würde ich den PR gerne schließen.

@kapsner
Copy link
Member Author

kapsner commented Oct 13, 2022

Hi @makampf, ja gerne in einem gemeinsamen Meeting nächsten Mo / Di zusammen mit @joundso und @sebmate in Kombi mit #38

@kapsner
Copy link
Member Author

kapsner commented Oct 13, 2022

Grundlegendes Ziel wäre es, #46 zu adressieren

@kapsner
Copy link
Member Author

kapsner commented Oct 18, 2022 •
edited
Loading

@makampf Kurzes Feedback zum heutigen Call: @sebmate wird das Thema Performance-Optimierung des FHIR-Gateway mit in die nächste ETL-Telko mitnehmen.
Grundsätzlich ist bisher nicht besprochen / genau überlegt, welcher Weg am besten geeignet ist, um die Performance-Optimierung (im Sinne von #46 ) zu adressieren.

Ideen wären:

  • Index im JSON setzen
  • neue Spalten (Pat-ID/Enc-ID/Zeitstempel)
  • Datenbank-Funktionen
  • ....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kapsner @makampf @renovate-bot