delta-io/delta

[Kernel][Testing] Add more thorough testing for DataType.toJson

Chiusa

#2313 aperta il 20 nov 2023

Vedi su GitHubGuida contributor
 (6 commenti) (0 reazioni) (0 assegnatari)Scala (2158 fork)batch import
enhancementgood first issuekernelstale

Metriche repository

Star
 (8947 stelle)
Metriche merge PR
 (Merge medio 7g 1h) (142 PR mergiate in 30 g)

Descrizione

Feature request

Which Delta project/connector is this regarding?

  • Spark
  • Standalone
  • Flink
  • Kernel
  • Other (fill in here)

Overview

All of our DataType classes provide toJson which serializes the type following the delta protocol schema serialization rules https://github.com/delta-io/delta/blob/master/PROTOCOL.md#schema-serialization-format. We use this method to serialize the physical and logical schemas in ScanStateRow.

toJson is only tested by the 2 round-trip tests in DefaultJsonHandlerSuite. We should add more thorough tests for toJson.

Guida contributor

Direzione di ricerca
Esamina il DefaultJsonHandlerSuite esistente per comprendere gli attuali test di round trip per toJson, quindi crea test unitari completi per ogni sottoclasse DataType che coprano vari scenari di serializzazione dello schema.
Tech stack
scala
Dominio
databasebackend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ScalaTesting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60