Docs: drivine4j doesn't read the flat DATABASE_* variables, as 02-connecting.md and design.md say

Aperta Adatta ai principianti
  • Qualcosa non va?
#361 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
kotlin, spring
Ambito
documentation

Direzione di ricerca

L'affermazione si trova in docs/02-connecting.md (righe 3-4) e in docs/design.md (intorno alla riga 125 e alle righe 129-131). Prima di modificare, verifica i dati su drivine4j in docs/04-quick-start.md e in src/main/kotlin/org/drivine/connection/DataSourceMap.kt. È completato quando entrambi i file indicano che le variabili piatte DATABASE_* appartengono a Drivine per TypeScript e che drivine4j usa le proprietà di Spring (database.datasources.<name>.type oppure DATABASE_DATASOURCES_<NAME>_TYPE), quindi un singolo .env non copre ancora la JVM.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

documentation

docs/02-connecting.md says the environment variables are the same as Drivine for TypeScript's "and drivine4j's, so one .env serves an application in any of them" (https://github.com/liberation-data/drivine-python/blob/64799691075093625a09db9ec5442ae7c2d498b6/docs/02-connecting.md#L3-L4). docs/design.md says the same of "TypeScript and the JVM" (https://github.com/liberation-data/drivine-python/blob/64799691075093625a09db9ec5442ae7c2d498b6/docs/design.md#L125 and #L129-L131).

That holds for Drivine for TypeScript, which reads DATABASE_TYPE, DATABASE_HOST and the rest, and <PREFIX>_DATABASE_* for a second database (https://github.com/liberation-data/drivine/blob/93974ae499e8b3d283e0fdbc2265ca36ae560f11/src/connection/ConnectionProperties.ts#L61-L68). It doesn't hold for drivine4j: it reads no flat DATABASE_* variables. Its datasources are Spring properties under database.datasources.<name> (https://github.com/liberation-data/drivine4j/blob/8bb9acd5f93fa5b8c7d77df6ce634ee07e16942e/docs/04-quick-start.md#L9-L27), which Spring's relaxed binding takes from the environment as DATABASE_DATASOURCES_<NAME>_TYPE, DATABASE_DATASOURCES_<NAME>_HOST and so on, or from a DataSourceMap built in code (https://github.com/liberation-data/drivine4j/blob/8bb9acd5f93fa5b8c7d77df6ce634ee07e16942e/src/main/kotlin/org/drivine/connection/DataSourceMap.kt).

Fix

Say that the variables are Drivine for TypeScript's, and that drivine4j takes the same settings as Spring properties (database.datasources.<name>.type, or DATABASE_DATASOURCES_<NAME>_TYPE in the environment), so one .env serves Python and Node but not the JVM as it stands. In both files.

Lingua principale
Python
Stelle
1
Fork
0
Merge medio
1h 53m
PR unite (30g)
157

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di liberation-data/drivine-python

Tutte le issue di liberation-data/drivine-python

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.