Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Describe schemas in the DSL (#13) and Tool to infer initial schema (#12) #72

Closed
wants to merge 2 commits into from

Conversation

MShahzaib
Copy link
Contributor

feat: Describe schemas in the DSL (#13) and Tool to infer the initial dataset schema (#12)

Closes #[13], #[12].

Summary of Changes

  • added "schema" declaration to grammar for describing a dataset's initial schema
  • added ".sdsschema" extension for schemas
  • added Tablesaw for inferring the initial dataset schema
  • implemented schema inference for the initial dataset schema with a Unit test

@github-actions
Copy link

github-actions bot commented Jun 22, 2022

MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ CREDENTIALS secretlint yes no 2.05s
✅ GIT git_diff yes no 0.01s
✅ JSON eslint-plugin-jsonc 1 0 0 1.01s
✅ JSON jsonlint 1 0 0.23s
✅ JSON prettier 1 0 0 0.48s
✅ JSON v8r 1 0 4.15s
✅ KOTLIN ktlint 14 0 0 27.22s
✅ MARKDOWN markdownlint 1 0 0 0.38s
✅ MARKDOWN markdown-link-check 1 0 0.45s
✅ MARKDOWN markdown-table-formatter 1 0 0 0.32s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

@lars-reimann
Copy link
Member

@MShahzaib For the sake of easier review, please split this into two PRs. The first one should only add the syntax for schemas (#13), and the second should only introduce the schema inference (#12).

@MShahzaib
Copy link
Contributor Author

@lars-reimann okay i will do that !

@MShahzaib MShahzaib closed this Jun 23, 2022
@lars-reimann lars-reimann deleted the feat_Describe_and_infer_schema branch January 30, 2023 13:49
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