delta-io/delta

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

クローズ

#2,313 opened on 2023/11/20

 (6 件のコメント) (0 件のリアクション) (0 人の担当者)Scala (2,154 件のフォーク)batch import
enhancementgood first issuekernelstale

Repository metrics

Stars
 (8,944 個のスター)
PR merge metrics
 (平均マージ 7d 1h) (30d で 142 merged PRs)

説明

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.

コントリビューターガイド