delta-io/delta

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

Open

#2 313 ouverte le 20 nov. 2023

Voir sur GitHub
 (5 commentaires) (0 réactions) (0 assignés)Scala (2 100 forks)batch import
enhancementgood first issuekernel

Métriques du dépôt

Stars
 (8 807 stars)
Métriques de merge PR
 (Merge moyen 7j 1h) (142 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez le DefaultJsonHandlerSuite existant pour comprendre les tests d'aller retour actuels pour toJson, puis créez des tests unitaires complets pour chaque sous classe DataType couvrant divers scénarios de sérialisation de schéma.
Stack technique
scala
Domaine
databasebackend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
ScalaTesting
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60