You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by GitBox <gi...@apache.org> on 2022/08/19 14:58:45 UTC

[GitHub] [arrow-nanoarrow] paleolimbot opened a new pull request, #25: Fix memcheck in CI (and memcheck errors in C)

paleolimbot opened a new pull request, #25:
URL: https://github.com/apache/arrow-nanoarrow/pull/25

   Fixes #22.


-- 
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: issues-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-nanoarrow] paleolimbot merged pull request #25: Fix memcheck in CI (and memcheck errors in C)

Posted by GitBox <gi...@apache.org>.
paleolimbot merged PR #25:
URL: https://github.com/apache/arrow-nanoarrow/pull/25


-- 
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: issues-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-nanoarrow] codecov-commenter commented on pull request #25: Fix memcheck in CI (and memcheck errors in C)

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #25:
URL: https://github.com/apache/arrow-nanoarrow/pull/25#issuecomment-1220837816

   # [Codecov](https://codecov.io/gh/apache/arrow-nanoarrow/pull/25?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#25](https://codecov.io/gh/apache/arrow-nanoarrow/pull/25?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (5653f00) into [main](https://codecov.io/gh/apache/arrow-nanoarrow/commit/a78c809e48606d00c2de036691e6016da09c614d?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (a78c809) will **increase** coverage by `0.20%`.
   > The diff coverage is `100.00%`.
   
   ```diff
   @@            Coverage Diff             @@
   ##             main      #25      +/-   ##
   ==========================================
   + Coverage   91.66%   91.87%   +0.20%     
   ==========================================
     Files          12       12              
     Lines        1296     1304       +8     
     Branches       46       47       +1     
   ==========================================
   + Hits         1188     1198      +10     
   + Misses         75       74       -1     
   + Partials       33       32       -1     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/arrow-nanoarrow/pull/25?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [src/nanoarrow/bitmap\_inline.h](https://codecov.io/gh/apache/arrow-nanoarrow/pull/25/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL25hbm9hcnJvdy9iaXRtYXBfaW5saW5lLmg=) | `100.00% <ø> (ø)` | |
   | [src/nanoarrow/array.c](https://codecov.io/gh/apache/arrow-nanoarrow/pull/25/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL25hbm9hcnJvdy9hcnJheS5j) | `89.39% <100.00%> (+0.38%)` | :arrow_up: |
   | [src/nanoarrow/array\_view.c](https://codecov.io/gh/apache/arrow-nanoarrow/pull/25/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL25hbm9hcnJvdy9hcnJheV92aWV3LmM=) | `91.77% <100.00%> (+0.05%)` | :arrow_up: |
   | [src/nanoarrow/schema.c](https://codecov.io/gh/apache/arrow-nanoarrow/pull/25/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c3JjL25hbm9hcnJvdy9zY2hlbWEuYw==) | `81.02% <0.00%> (+0.72%)` | :arrow_up: |
   
   :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   


-- 
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: issues-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-nanoarrow] lidavidm commented on pull request #25: Fix memcheck in CI (and memcheck errors in C)

Posted by GitBox <gi...@apache.org>.
lidavidm commented on PR #25:
URL: https://github.com/apache/arrow-nanoarrow/pull/25#issuecomment-1220792414

   Ah…seems it's probably the suppressions not taking effect


-- 
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: issues-unsubscribe@arrow.apache.org

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