Skip to content

Add -static flag - #7292

Open
mattnibs wants to merge 1 commit into
mainfrom
static-flag
Open

Add -static flag#7292
mattnibs wants to merge 1 commit into
mainfrom
static-flag

Conversation

@mattnibs

@mattnibs mattnibs commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

This commit adds the -static flag which when enabled, forces static type checking on all input files. By default csup and parquet are statically checked- all other file types are dynamic. If the -static flag is enabled the entire file is read and the full type information is gathered. Non-seekable inputs (e.g., /dev/stdin) cannot be statically checked an will return an error -static is enabled.

This commit removes the -samplesize flag.

This commit adds the -static flag which when enabled, forces static type
checking on all input files. By default csup and parquet are statically
checked- all other file types are dynamic. If the -static flag is enabled
the entire file is read and the full type information is gathered.
Non-seekable inputs (e.g., /dev/stdin) cannot be statically checked an will
return an error -static is enabled.

This commit removes the -samplesize flag.
@mattnibs
mattnibs marked this pull request as ready for review September 8, 2026 22:54
@philrz
philrz requested a review from a team September 8, 2026 23:42
@philrz

philrz commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

One small finding: Right now it's possible to invoke -static and -dynamic simultaneously. It looks like -dynamic is "winning" at the moment, but some kind of guard to prevent this might be helpful.

$ echo '{"a": "b"}' > data.json &&
  super -version &&
  super -static -dynamic -c "values foo" data.json

Version: v0.3.0-379-gd55bada8f

error({message:"no such field foo",on:{}})

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