Josh Pschorr

Results 42 issues of Josh Pschorr

Fixes #495 Adds `TUPLEUNION` and `TUPLEMERGE` functions `TUPLEUNION` is as per 6.3.2 of the spec (https://partiql.org/assets/PartiQL-Specification.pdf#subsubsection.6.3.2) `TUPLEMERGE` is inspired by description of concatenating binding environments as per section 3.3 of...

`ParserError`, `AstTransformationError`, `PlanErr`, and `EvalErr` should implement `Error` (and perhaps `miette::Diagnostic`?) _Inspired by @am357 in https://github.com/partiql/partiql-lang-rust/pull/496#discussion_r1765421366_

I had issues practically using Shapes with Ids in an external project. I began fixing one thing, and ended up with this refactor. In addition to these changes, I also...

After #485 `PartiqlShapeBuilder.any_of`: ``` pub fn any_of(&self, types: I) -> PartiqlShape where I: IntoIterator, { let any_of = AnyOf::from_iter(types); match any_of.types.len() { 0 => type_dynamic!(), 1 => { let AnyOf...

I found this a bit surprising: ```rust #[test] fn ion_dates() { let l = "2000T"; let r = "2000-01-01T00:00:00.000+00:00"; let l = ion_rs::Element::read_one(l).expect("read"); let r = ion_rs::Element::read_one(r).expect("read"); let reencode_txt =...

bug

*Issue #, if available:* *Description of changes:* By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

*Issue #, if available:* *Description of changes:* By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

- [x] Changelog - https://github.com/partiql/partiql-lang-rust/pull/548#discussion_r2008297427 - [ ] Revisit parsing number enforcement for e.g., `SHORTEST k` - https://github.com/partiql/partiql-lang-rust/pull/548#discussion_r2008289953 - [ ] Revisit parsing/pretty-printing multiple/alternative labels - https://github.com/partiql/partiql-lang-rust/pull/548#discussion_r2008272580

Add better type-specific pretty printing for embedded documents. _Originally posted by @jpschorr in https://github.com/partiql/partiql-lang-rust/pull/507#discussion_r1805217916_

> Does it also need to include `isNullCall` and `isMissingCall` as `partiql-lang-kotlin` [implements](https://github.com/partiql/partiql-lang-kotlin/blob/da22cf8665e1dcf0b3747452285cca6df73c0e95/partiql-spi/src/main/kotlin/org/partiql/spi/function/Function.kt#L28-L29)? _Originally posted by @am357 in https://github.com/partiql/partiql-lang-rust/pull/496#discussion_r1783620523_