[V1] Defines source and sink APIs for PartiQL data streams. #1655
Conversation
| @NotNull | ||
| BigDecimal readDecimal(); | ||
|
|
There was a problem hiding this comment.
missing
@NotNull
BigDecimal readNumeric();
| void readNull(); | ||
|
|
||
| void readMissing(); |
There was a problem hiding this comment.
likely not necessary as these have no effect
| /** | ||
| * Positions the source to the next value, return its type. | ||
| */ | ||
| PType next(); |
There was a problem hiding this comment.
Annotate as nullable
CROSS-ENGINE-REPORT ❌
Testing Details
Result Details
Now FAILING Tests ❌The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact. Now IGNORED Tests ❌The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact. Now Passing Tests176 test(s) were previously failing in BASE (LEGACY-V0.14.8) but now pass in TARGET (EVAL-FA2CB2C). Before merging, confirm they are intended to pass. The complete list can be found in GitHub CI summary, either from Step Summary or in the Artifact. CROSS-COMMIT-REPORT ❌
Testing DetailsResult Details
Now FAILING Tests ❌The following 1 test(s) were previously PASSING in BASE but are now FAILING in TARGET: Click here to see
|
|
waiting until datetime PR #1657 is in and rebase |
|
Replaced by #1687 |
1 participant
Description
This PR adds a source and sink API to the SPI package which will become the backing interfaces for various PartiQL encoding schemes (firstly Ion). These APIs are just like the reader/writer APIs from IonJava, but are intended to be extended to get values into a PartiQL system. The primary/default application here will be to encode PartiQL data in typed Ion.
See #1654
Other Information
and Code Style Guidelines? YES
License Information
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.