You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by "pdxcodemonkey (GitHub)" <gi...@apache.org> on 2019/01/07 21:25:32 UTC

[GitHub] [geode-native] pdxcodemonkey opened pull request #431: GEODE-6249: port attributes mutator test

... and rename it "ExpirationTest," since it doesn't actually use/test AttributeMutator.

[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.04%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
+ Coverage    72.17%   72.21%   +0.04%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
+ Hits         37803    37806       +3     
+ Misses       14577    14543      -34
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `51.94% <0%> (-0.42%)` | :arrow_down: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.48% <0%> (+0.69%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `34.79% <0%> (+0.73%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.17% <0%> (+0.99%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/statistics/StatArchiveWriter.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvU3RhdEFyY2hpdmVXcml0ZXIuY3Bw) | `83.09% <0%> (+1.45%)` | :arrow_up: |
| [cppcache/src/TcrPoolEndPoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjclBvb2xFbmRQb2ludC5jcHA=) | `85.13% <0%> (+5.4%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.11%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
+ Coverage    72.17%   72.28%   +0.11%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
+ Hits         37803    37843      +40     
+ Misses       14577    14506      -71
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/ThinClientLocatorHelper.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRMb2NhdG9ySGVscGVyLmNwcA==) | `73.98% <0%> (+0.4%)` | :arrow_up: |
| [cppcache/src/ThinClientRegion.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWdpb24uY3Bw) | `55.65% <0%> (+0.45%)` | :arrow_up: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `53.19% <0%> (+0.83%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.69% <0%> (+0.89%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/RemoteQuery.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1JlbW90ZVF1ZXJ5LmNwcA==) | `76.84% <0%> (+1.05%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `35.16% <0%> (+1.09%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.31% <0%> (+1.13%)` | :arrow_up: |
| ... and [4 more](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
+ Coverage    72.17%   72.29%   +0.12%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
+ Hits         37803    37847      +44     
+ Misses       14577    14502      -75
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/ThinClientLocatorHelper.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRMb2NhdG9ySGVscGVyLmNwcA==) | `73.98% <0%> (+0.4%)` | :arrow_up: |
| [cppcache/src/ThinClientRegion.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWdpb24uY3Bw) | `55.65% <0%> (+0.45%)` | :arrow_up: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `53.19% <0%> (+0.83%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.69% <0%> (+0.89%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/RemoteQuery.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1JlbW90ZVF1ZXJ5LmNwcA==) | `76.84% <0%> (+1.05%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `35.16% <0%> (+1.09%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.31% <0%> (+1.13%)` | :arrow_up: |
| ... and [5 more](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] pdxcodemonkey closed pull request #431: GEODE-6249: port attributes mutator test

Posted by "pdxcodemonkey (GitHub)" <gi...@apache.org>.
[ pull request closed by pdxcodemonkey ]

[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.13%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           develop    #431      +/-   ##
==========================================
+ Coverage    72.17%   72.3%   +0.13%     
==========================================
  Files          655     654       -1     
  Lines        52380   52349      -31     
==========================================
+ Hits         37803   37851      +48     
+ Misses       14577   14498      -79
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/ThinClientLocatorHelper.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRMb2NhdG9ySGVscGVyLmNwcA==) | `73.98% <0%> (+0.4%)` | :arrow_up: |
| [cppcache/src/ThinClientRegion.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWdpb24uY3Bw) | `55.65% <0%> (+0.45%)` | :arrow_up: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `53.19% <0%> (+0.83%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.76% <0%> (+0.96%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/RemoteQuery.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1JlbW90ZVF1ZXJ5LmNwcA==) | `76.84% <0%> (+1.05%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `35.16% <0%> (+1.09%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.31% <0%> (+1.13%)` | :arrow_up: |
| ... and [6 more](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **decrease** coverage by `0.02%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
- Coverage    72.17%   72.14%   -0.03%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
- Hits         37803    37765      -38     
- Misses       14577    14584       +7
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/ThinClientPoolDM.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uaHBw) | `52.45% <0%> (-1.64%)` | :arrow_down: |
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `51.11% <0%> (-1.25%)` | :arrow_down: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `62.03% <0%> (-0.15%)` | :arrow_down: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `67.93% <0%> (+0.13%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.11%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
+ Coverage    72.17%   72.28%   +0.11%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
+ Hits         37803    37841      +38     
+ Misses       14577    14508      -69
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/ThinClientRegion.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWdpb24uY3Bw) | `55.65% <0%> (+0.45%)` | :arrow_up: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `53.19% <0%> (+0.83%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.69% <0%> (+0.89%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/RemoteQuery.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1JlbW90ZVF1ZXJ5LmNwcA==) | `76.84% <0%> (+1.05%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `35.16% <0%> (+1.09%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.31% <0%> (+1.13%)` | :arrow_up: |
| [cppcache/src/statistics/StatArchiveWriter.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvU3RhdEFyY2hpdmVXcml0ZXIuY3Bw) | `83.09% <0%> (+1.45%)` | :arrow_up: |
| ... and [3 more](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
+ Coverage    72.17%   72.29%   +0.12%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
+ Hits         37803    37847      +44     
+ Misses       14577    14502      -75
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/ThinClientLocatorHelper.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRMb2NhdG9ySGVscGVyLmNwcA==) | `73.98% <0%> (+0.4%)` | :arrow_up: |
| [cppcache/src/ThinClientRegion.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWdpb24uY3Bw) | `55.65% <0%> (+0.45%)` | :arrow_up: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `53.19% <0%> (+0.83%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.69% <0%> (+0.89%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/RemoteQuery.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1JlbW90ZVF1ZXJ5LmNwcA==) | `76.84% <0%> (+1.05%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `35.16% <0%> (+1.09%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.31% <0%> (+1.13%)` | :arrow_up: |
| ... and [5 more](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.12%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
+ Coverage    72.17%   72.29%   +0.12%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
+ Hits         37803    37847      +44     
+ Misses       14577    14502      -75
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/ThinClientLocatorHelper.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRMb2NhdG9ySGVscGVyLmNwcA==) | `73.98% <0%> (+0.4%)` | :arrow_up: |
| [cppcache/src/ThinClientRegion.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWdpb24uY3Bw) | `55.65% <0%> (+0.45%)` | :arrow_up: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `53.19% <0%> (+0.83%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.69% <0%> (+0.89%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/RemoteQuery.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1JlbW90ZVF1ZXJ5LmNwcA==) | `76.84% <0%> (+1.05%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `35.16% <0%> (+1.09%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.31% <0%> (+1.13%)` | :arrow_up: |
| ... and [5 more](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree-more) | |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] pdxcodemonkey commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "pdxcodemonkey (GitHub)" <gi...@apache.org>.
Yeah we'll need to check and see if we have coverage of attributes elsewhere,  cause this test definitely did/does nothing the original name would suggest.

[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-native] codecov-io commented on issue #431: GEODE-6249: port attributes mutator test

Posted by "codecov-io (GitHub)" <gi...@apache.org>.
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=h1) Report
> Merging [#431](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=desc) into [develop](https://codecov.io/gh/apache/geode-native/commit/a816ac99cbbac557629686cb2542fdc74d464338?src=pr&el=desc) will **increase** coverage by `0.04%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/apache/geode-native/pull/431/graphs/tree.svg?width=650&token=plpAqoqGag&height=150&src=pr)](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree)

```diff
@@             Coverage Diff             @@
##           develop     #431      +/-   ##
===========================================
+ Coverage    72.17%   72.21%   +0.04%     
===========================================
  Files          655      654       -1     
  Lines        52380    52349      -31     
===========================================
+ Hits         37803    37806       +3     
+ Misses       14577    14543      -34
```


| [Impacted Files](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==) | `51.94% <0%> (-0.42%)` | :arrow_down: |
| [cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=) | `72.5% <0%> (+0.46%)` | :arrow_up: |
| [cppcache/src/ClientMetadata.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL0NsaWVudE1ldGFkYXRhLmNwcA==) | `62.96% <0%> (+0.61%)` | :arrow_up: |
| [cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw) | `68.48% <0%> (+0.69%)` | :arrow_up: |
| [cppcache/include/geode/ExceptionTypes.hpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW5jbHVkZS9nZW9kZS9FeGNlcHRpb25UeXBlcy5ocHA=) | `34.79% <0%> (+0.73%)` | :arrow_up: |
| [cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw) | `63.17% <0%> (+0.99%)` | :arrow_up: |
| [cppcache/src/statistics/HostStatSampler.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvSG9zdFN0YXRTYW1wbGVyLmNwcA==) | `47.15% <0%> (+1.03%)` | :arrow_up: |
| [cppcache/src/statistics/StatArchiveWriter.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL3N0YXRpc3RpY3MvU3RhdEFyY2hpdmVXcml0ZXIuY3Bw) | `83.09% <0%> (+1.45%)` | :arrow_up: |
| [cppcache/src/TcrPoolEndPoint.cpp](https://codecov.io/gh/apache/geode-native/pull/431/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjclBvb2xFbmRQb2ludC5jcHA=) | `85.13% <0%> (+5.4%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=footer). Last update [a816ac9...6279c9a](https://codecov.io/gh/apache/geode-native/pull/431?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).


[ Full content available at: https://github.com/apache/geode-native/pull/431 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org