Skip to content

Conversation

@aajtodd
Copy link
Contributor

@aajtodd aajtodd commented Mar 7, 2022

Issue #

N/A

Description of changes

Follow up to #537 with integration tests for the generated event stream serde code

  • Fix a few edge cases with serialization and deserialization
  • Cleanup

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@aajtodd aajtodd requested a review from a team as a code owner March 7, 2022 20:13
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 7, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 10 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@aajtodd aajtodd requested a review from ianbotsf March 7, 2022 20:15
@aajtodd aajtodd merged commit 59fc9dc into feat-event-streams Mar 9, 2022
@aajtodd aajtodd deleted the event-streams-r2 branch March 9, 2022 16:50
aajtodd added a commit that referenced this pull request Mar 9, 2022
* Bootstrap event streams (#537)

* fix event stream filtering

* add parsing of common headers

* refractor frame decoder into a Flow

* remove event stream operation filter from customizations

* refactore event stream parsing; implement rough deserialization codegen

* fix warning

* filter out event stream errors

* render deserialization for exception event stream messages

* inject http request signature into the execution context once known

* add support for chunked signing

* add encode transform for message stream

* inline signing config builder

* initial event stream serialize implementation

* fix compile issues

* disable wip integration tests

* suppress test; cleanup codegen

* Event Stream Codegen Tests (#542)

* Checkpoint Event Streams (#544)

* fix tests

* increase windows runner memory
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