apache/gravitino

[Improvement] Possible null pointer exception in CatalogRegister.java

Open

#4189 aperta il 18 lug 2024

Vedi su GitHub
 (14 commenti) (0 reazioni) (1 assegnatario)Java (887 fork)auto 404
good first issueimprovement

Metriche repository

Star
 (3058 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

What would you like to be improved?

In checkCatalogExist, failedException is set to null and then thrown which could cause a NPE if it is not set in the exception block.

How should we improve?

Move throw failedException; to the correct place in the catch block?

Guida contributor

Direzione di ricerca
Ispeziona il metodo checkCatalogExist in CatalogRegister.java e sposta throw failedException all'interno del blocco catch per prevenire potenziali NPE.
Tech stack
java
Dominio
backenddata
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75