Re: [PR] Add trace stress test suit [skywalking-banyandb]

2024-03-22 Thread via GitHub


wu-sheng merged PR #422:
URL: https://github.com/apache/skywalking-banyandb/pull/422


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Add trace stress test suit [skywalking-banyandb]

2024-03-22 Thread via GitHub


codecov-commenter commented on PR #422:
URL: 
https://github.com/apache/skywalking-banyandb/pull/422#issuecomment-2016244884

   ## 
[Codecov](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 Report
   Attention: Patch coverage is `66.7%` with `8 lines` in your changes are 
missing coverage. Please review.
   > Project coverage is 47.66%. Comparing base 
[(`cde0cc9`)](https://app.codecov.io/gh/apache/skywalking-banyandb/commit/cde0cc95e62339e700455ed79e3fd8b7913a48af?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 to head 
[(`1b5ac71`)](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?dropdown=coverage&src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   
   > :exclamation: Current head 1b5ac71 differs from pull request most recent 
head 1129c98. Consider uploading reports for the commit 1129c98 to get more 
accurate results
   
   | 
[Files](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 | Patch % | Lines |
   |---|---|---|
   | 
[banyand/stream/tstable.go](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-YmFueWFuZC9zdHJlYW0vdHN0YWJsZS5nbw==)
 | 0.00% | [6 Missing :warning: 
](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   | 
[banyand/stream/part.go](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-YmFueWFuZC9zdHJlYW0vcGFydC5nbw==)
 | 0.00% | [2 Missing :warning: 
](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
 |
   
   Additional details and impacted files
   
   
   ```diff
   @@Coverage Diff @@
   ## main #422  +/-   ##
   ==
   - Coverage   48.07%   47.66%   -0.42% 
   ==
 Files 175  175  
 Lines   2164621646  
   ==
   - Hits1040710317  -90 
   - Misses  1034810457 +109 
   + Partials  891  872  -19 
   ```
   
   
   
   
   
   [:umbrella: View full report in Codecov by 
Sentry](https://app.codecov.io/gh/apache/skywalking-banyandb/pull/422?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   
   :loudspeaker: Have feedback on the report? [Share it 
here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] Add trace stress test suit [skywalking-banyandb]

2024-03-22 Thread via GitHub


wu-sheng commented on PR #422:
URL: 
https://github.com/apache/skywalking-banyandb/pull/422#issuecomment-2016244425

   Please fix CI.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] Add trace stress test suit [skywalking-banyandb]

2024-03-22 Thread via GitHub


hanahmily opened a new pull request, #422:
URL: https://github.com/apache/skywalking-banyandb/pull/422

   Add tracee stress test suite and fix several flaws found by the suite: 
   
   1. Create a deep copy of the tag family map data.
   2. Decrease snapshot references in the get element query function.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org