Identify component factories by identifier string
@kevinwallimann ci sta già lavorando.
Dal 19/2/2020.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 28/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- scala
- Ambito
- backend, developer-experience
Direzione di ricerca
Inizia leggendo ComponentFactory e ClassLoaderUtils, oltre al lavoro SPI a cui si fa riferimento in #83. Il compito consiste nell'aggiungere getIdentifier con il nome della classe come valore predefinito, quindi modificare la ricerca delle factory in modo che trovi tramite identificatore le factory fornite da ServiceLoader. Il lavoro è completato quando le configurazioni esistenti basate sul nome della classe continuano a funzionare e le configurazioni possono invece usare un identificatore stabile.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
Currently, component factories are loaded in ClassLoaderUtils given their fully qualified classnames. The classname is passed by the configuration. (e.g. component.writer)
That means that components don't have the possibility to be refactored (renaming, moving to a different package) without introducing a breaking change which would require updating any existing configuration that uses that component.
Tasks
- Add a method
getIdentifier: Stringto the interfaceComponentFactory.getClass.getNamemay be used as a default value. (so this feature won't be a breaking change) - Implementing components are responsible for providing a unique identifier. It's advisable to prefix the identifier with a human readable name, because it will be referenced in the configuration, logged, etc..
- Use
getIdentifierto load the factory inClassLoaderUtils. Currently, it loads the class directly given the class name. This approach doesn't work to efficiently load the factory by the identifier. With #83, component factories can be loaded using the Service Provider Interface (SPI), i.e. withServiceLoader. All factories expose thegetIdentifiermethod, that's how it can be found
Other
- The same identifier might be used by each component to prefix its configuration properties to avoid name clashes.
- Lingua principale
- Scala
- Stelle
- 47
- Fork
- 14
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di AbsaOSS/hyperdrive
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 45/100
AbsaOSS/hyperdrive#269 ·
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
AbsaOSS/hyperdrive#240 ·
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
AbsaOSS/hyperdrive#239 ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
AbsaOSS/hyperdrive#230 ·
-
Atum integration Aperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 20/100
AbsaOSS/hyperdrive#211 · 1 commento · 1 reazione · 1 assegnatario ·
Tutte le issue di AbsaOSS/hyperdrive
Issue simili
-
RS_PixelAsPoint raises out of grid where RS_PixelAsCentroid and RS_PixelAsPolygon extrapolate Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Area: Excel support
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
orbeon/orbeon-forms#7893 ·
-
x:action/fix x:module/practice-exercise x:size/small
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
lichess-org/lila#21788 · 2 commenti ·
-
area:Iceberg area:writer documentation requires-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
apache/datafusion-comet#6147 ·