Skip to content

Closer match options for FracturedJson - #2878

Merged
lemire merged 3 commits into
simdjson:masterfrom
maxbachmann:master
Sep 24, 2026
Merged

lemire merged 3 commits into
simdjson:masterfrom
maxbachmann:master

Conversation

@maxbachmann

Copy link
Copy Markdown
Contributor

This makes the following changes:

  • removes max_inline_length (was dropped in FracturedJson in favor of the total line length)
  • removes max items per line (I don't think FracturedJson ever had this option. Maybe was a stop gap before we properly handled line lengths)
  • added nested bracket padding
  • added always expand depth

We do have two options with different defaults from FracturedJson

  • simple_bracket_padding true vs false
  • max_compact_array_complexity 1 vs 2

In part this PR is so there can be review on changing these config options since in the table handling rework we will probably want to drop a couple more options:

  • min_table_rows
  • table_similarity_threshold

and introduce

  • max_table_row_complexity
  • min_compact_array_row_items

@lemire

lemire commented Sep 23, 2026

Copy link
Copy Markdown
Member

Running tests

@lemire
lemire merged commit 237e634 into simdjson:master Sep 24, 2026
115 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants