Skip to content

HTTP-202 Refactor and fix test#211

Merged
Grzegorz Kołakowski (grzegorz8) merged 2 commits intogetindata:mainfrom
davidradl:http202-2
Jan 19, 2026
Merged

HTTP-202 Refactor and fix test#211
Grzegorz Kołakowski (grzegorz8) merged 2 commits intogetindata:mainfrom
davidradl:http202-2

Conversation

@davidradl
Copy link
Copy Markdown
Contributor

Description

Refactored the implmentation of the deserilization change added list release - in response to feedback on the [Apache Port](Change deserialize method so it can work with Flink 2). I also noticed the test was not testing the code that was added, so I have amended the test.

Resolves HTTP-202

PR Checklist

Signed-off-by: davidradl <david_radley@uk.ibm.com>
@davidradl David Radley (davidradl) changed the title http202 refactor and fix test HTTP202 refactor and fix test Jan 9, 2026
Signed-off-by: davidradl <david_radley@uk.ibm.com>
@davidradl
Copy link
Copy Markdown
Contributor Author

Grzegorz Kołakowski (@grzegorz8) thanks for the approve - please could you merge as well?

@grzegorz8 Grzegorz Kołakowski (grzegorz8) changed the title HTTP202 refactor and fix test HTTP-202 Refactor and fix test Jan 19, 2026
@grzegorz8 Grzegorz Kołakowski (grzegorz8) merged commit 7a29b37 into getindata:main Jan 19, 2026
3 checks passed
@grzegorz8
Copy link
Copy Markdown
Member

David Radley (@davidradl) Merged.

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