Fix C++ SDK core dump issue #2
Merged
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
Signed-off-by: lizhanhui <[email protected]>
Signed-off-by: Zhanhui Li <[email protected]>
… is not formattable Signed-off-by: Li Zhanhui <[email protected]>
lizhanhui
added a commit
that referenced
this pull request
Mar 25, 2024
* fix: sync namespace from server Settings * feat: use opentelemetry for tracing/metrics/logging * Remove broken links and add targets to generate compile_commands.json Signed-off-by: lizhanhui <[email protected]> * fix: timer task may invoke a call to a destructing stream Signed-off-by: Zhanhui Li <[email protected]> * fix: update document as we have changed the way to generate compile_commands.json * fix: static_cast StreamState to std::uint8_t as enum class by default is not formattable Signed-off-by: Li Zhanhui <[email protected]> --------- Signed-off-by: lizhanhui <[email protected]> Signed-off-by: Zhanhui Li <[email protected]> Signed-off-by: Li Zhanhui <[email protected]>
lizhanhui
added a commit
that referenced
this pull request
Jul 17, 2024
* feat: add tagged-release github action pipeline Signed-off-by: Zhanhui Li <[email protected]> * Fix C++ SDK core dump issue (#2) * fix: sync namespace from server Settings * feat: use opentelemetry for tracing/metrics/logging * Remove broken links and add targets to generate compile_commands.json Signed-off-by: lizhanhui <[email protected]> * fix: timer task may invoke a call to a destructing stream Signed-off-by: Zhanhui Li <[email protected]> * fix: update document as we have changed the way to generate compile_commands.json * fix: static_cast StreamState to std::uint8_t as enum class by default is not formattable Signed-off-by: Li Zhanhui <[email protected]> --------- Signed-off-by: lizhanhui <[email protected]> Signed-off-by: Zhanhui Li <[email protected]> Signed-off-by: Li Zhanhui <[email protected]> * fix: fix stream state transition with gRPC reactor Signed-off-by: Li Zhanhui <[email protected]> * fix: revamp TelemetryBidiRecator Signed-off-by: Li Zhanhui <[email protected]> * feat: explicitly control exported symbols Signed-off-by: Li Zhanhui <[email protected]> * fix: release write hold when OnReadDone with ok=false and there is no inflight write Signed-off-by: Li Zhanhui <[email protected]> * feat: revamp TelemetryBidiReactor states and their transition graph Signed-off-by: Li Zhanhui <[email protected]> * fix: example publish message QPS stats lambda * Fifo opt (apache#732) * Prepare to optimize FIFO publishing Signed-off-by: Li Zhanhui <[email protected]> * fix: SendReceipt now contains std::unique_ptr<Message> being sent Signed-off-by: Li Zhanhui <[email protected]> * fix: add doc explaining why we taking ownership of the message being sent Signed-off-by: Li Zhanhui <[email protected]> * feat: implement FifoProducerPartition Signed-off-by: Li Zhanhui <[email protected]> * feat: implement FifoProducerImpl Signed-off-by: Li Zhanhui <[email protected]> * feat: implement builder for FifoProducer Signed-off-by: Li Zhanhui <[email protected]> * fix: prepare to debug Signed-off-by: Li Zhanhui <[email protected]> * fix: log sending sending stages Signed-off-by: Li Zhanhui <[email protected]> --------- Signed-off-by: Li Zhanhui <[email protected]> * fix: spell errors Signed-off-by: Li Zhanhui <[email protected]> * fix: markdown code blocks should be fenced by blank lines Signed-off-by: Li Zhanhui <[email protected]> * fix: copyright header Signed-off-by: Li Zhanhui <[email protected]> * fix: copyright header Signed-off-by: Li Zhanhui <[email protected]> * fix: include statements Signed-off-by: Zhanhui Li <[email protected]> * feat: build example_fifo_producer in BUILD.bazel Signed-off-by: Zhanhui Li <[email protected]> * fix: apache#713 correct misuse of absl::make_optional Signed-off-by: Zhanhui Li <[email protected]> --------- Signed-off-by: Zhanhui Li <[email protected]> Signed-off-by: lizhanhui <[email protected]> Signed-off-by: Li Zhanhui <[email protected]>
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.
Which Issue(s) This PR Fixes
Fixes #issue_id
Brief Description
How Did You Test This Change?