drill
DRILL-5955: Revisit Union Vectors
#2543
Open

DRILL-5955: Revisit Union Vectors #2543

vdiravka wants to merge 1 commit into apache:master from vdiravka:DRILL-5955
vdiravka
vdiravka3 years ago

DRILL-5955: Revisit Union Vectors

Description

Draft PR for support Union type for JSON2

  • Enabled Union/Variant vectors for JSON2
  • fix validation for Union vectors
  • fix List vector in the Union vector
  • check all existing tests work fine

Documentation

TBA

Testing

existing tests will be enabled for JSON2

vdiravka DRILL-5955: Revisit Union Vectors
cb8eb445
cgivre
cgivre3 years ago👍 1

@vdiravka This is looking good. Quick question, can we add a configuration variable to the JSON Format to enable/disable this at the plugin instance level?

cgivre cgivre added enhancement
cgivre cgivre added doc-impacting
cgivre cgivre added json

Login to write a write a comment.

Login via GitHub

Reviewers
No reviews
Assignees
No one assigned
Labels
Milestone