Full project review based on latest version of JNet suite #193
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…rm-to-the-right-namespaces
… enhanced serializer/deserializer (#182) * #92: update to JNet 1.5.5 * #92: full class review due to breaking change in JCOBridge * #175: improvements in Consumer/Producer Builders * Description review * #175: review classes to accept new KNetCompactedReplicator * #175: added serialization projects * #175: moved to Java SerDes due to error in C# compilation within container * #175: review of KNetConsumer, KNetProducer and KNetSerDes; added specific test for KNetConsumer and KNetProducer * #175: update serialization and added MessagePack type * #175: updates on sync management * #175: documentation update
* #121: version update to 1.5.4 * Added missing SourceLink
…m-to-the-right-namespaces waiting for masesgroup/JNet#178
…ed into Java types
masesdevelopers
added
documentation
masesdevelopers
deleted the
168-review-all-classes-to-conform-to-the-right-namespaces
branch
9 tasks
masesdevelopers
added a commit
that referenced
this pull request
Jul 7, 2023
* Full project review based on latest version of JNet suite (#193) * #168: first step: replace namespaces in all available classes * #93: documentation alignment * Added a benchmark to verify performance roundtrip (#178) * #53: added new roundtrip benchmark * #53: update performance description file * #168: moved KNet specific classes into dedicated folder * #179: fixed compacted topic creation (#180) * #92: full class review due to breaking change in JCOBridge * #168: first generation * Added KNetCompactedReplicator, evolved KNetProducer and KNetConsumer, enhanced serializer/deserializer (#182) * #92: full class review due to breaking change in JCOBridge * #175: improvements in Consumer/Producer Builders * #175: review classes to accept new KNetCompactedReplicator * #175: added serialization projects * #175: moved to Java SerDes due to error in C# compilation within container * #175: review of KNetConsumer, KNetProducer and KNetSerDes; added specific test for KNetConsumer and KNetProducer * #175: update serialization and added MessagePack type * #175: updates on sync management * #175: documentation update * Update documentation after commit fb2bded * Added missing SourceLink * Configuration is now managed using a JSON file (masesgroup/JNet#179) * Update configuration and files * Temporary commit: many classes shall be removed because are old * #185: fix .NET Framework PowerShell version (#186) * #92: update to JNet 2.0.0.0 * #121: update to version 2.0.0.0 * Correction on namespace * Update classes after JNetReflector update for masesgroup/JNet#195 * #168 (comment): implementation of special listeners * Reviewed implementation of KNet version of ConnectStandalone and ConnectDistributed * #88: full update to Apache Kafka 3.5.0 * Code alignment to latest JNetReflector: nullable native types converted into Java types * Update workflows to avoid documentation generation out of main branch build * Update documentation after commit 312b4bf * Added missing documentation (#194) * #24, #168: review documentation, removed unused classes, KafkaClientSupplier becomes a listener * #24: fix documentation location (#195) * #24: removed many warning from workflows output (#196) * #24: added disclaimer for version 2.0.0 (#197) * Update documentation after commit cb755ee * V2 merge conflicts solved (#198) * Remerge (#199) * #24: added some documentation for serializer/deserializer (#201) * Update documentation after commit b9e9db3 --------- Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR updates KNet in many areas:
NOTE: mainly due to changes in namespaces the PR introduces breaking changes
Related Issue
Closed #168
Fix #92
Fix #121
Fix #88
Motivation and Context
How Has This Been Tested?
Some local tests based on available tests
Screenshots (if appropriate):
Types of changes
Checklist: