[
https://issues.apache.org/jira/browse/GEODE-8553?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17212011#comment-17212011
]
ASF GitHub Bot commented on GEODE-8553:
---------------------------------------
codecov-io commented on pull request #660:
URL: https://github.com/apache/geode-native/pull/660#issuecomment-706753854
# [Codecov](https://codecov.io/gh/apache/geode-native/pull/660?src=pr&el=h1)
Report
> Merging
[#660](https://codecov.io/gh/apache/geode-native/pull/660?src=pr&el=desc) into
[develop](https://codecov.io/gh/apache/geode-native/commit/5f06ae2044004397492249fcf13bcd90e75b955b?el=desc)
will **decrease** coverage by `0.07%`.
> The diff coverage is `93.12%`.
[](https://codecov.io/gh/apache/geode-native/pull/660?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## develop #660 +/- ##
===========================================
- Coverage 74.03% 73.95% -0.08%
===========================================
Files 644 644
Lines 51133 51030 -103
===========================================
- Hits 37854 37740 -114
- Misses 13279 13290 +11
```
| [Impacted
Files](https://codecov.io/gh/apache/geode-native/pull/660?src=pr&el=tree) |
Coverage Δ | |
|---|---|---|
|
[cppcache/src/ThinClientLocatorHelper.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRMb2NhdG9ySGVscGVyLmNwcA==)
| `91.15% <92.85%> (+5.04%)` | :arrow_up: |
|
[cppcache/src/ThinClientLocatorHelper.hpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRMb2NhdG9ySGVscGVyLmhwcA==)
| `100.00% <100.00%> (ø)` | |
|
[cppcache/src/ThinClientPoolDM.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRQb29sRE0uY3Bw)
| `75.46% <100.00%> (-0.63%)` | :arrow_down: |
|
[...tion-test/testThinClientRemoteQueryFailoverPdx.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvaW50ZWdyYXRpb24tdGVzdC90ZXN0VGhpbkNsaWVudFJlbW90ZVF1ZXJ5RmFpbG92ZXJQZHguY3Bw)
| `85.48% <0.00%> (-4.04%)` | :arrow_down: |
|
[cppcache/src/TXCommitMessage.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RYQ29tbWl0TWVzc2FnZS5jcHA=)
| `74.50% <0.00%> (-1.97%)` | :arrow_down: |
|
[cppcache/src/TcrEndpoint.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckVuZHBvaW50LmNwcA==)
| `54.55% <0.00%> (-1.83%)` | :arrow_down: |
|
[cppcache/src/RemoteQuery.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1JlbW90ZVF1ZXJ5LmNwcA==)
| `75.53% <0.00%> (-1.07%)` | :arrow_down: |
|
[cppcache/src/ThinClientRedundancyManager.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RoaW5DbGllbnRSZWR1bmRhbmN5TWFuYWdlci5jcHA=)
| `75.31% <0.00%> (-0.47%)` | :arrow_down: |
|
[cppcache/src/TcrConnection.cpp](https://codecov.io/gh/apache/geode-native/pull/660/diff?src=pr&el=tree#diff-Y3BwY2FjaGUvc3JjL1RjckNvbm5lY3Rpb24uY3Bw)
| `72.79% <0.00%> (-0.16%)` | :arrow_down: |
------
[Continue to review full report at
Codecov](https://codecov.io/gh/apache/geode-native/pull/660?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/660?src=pr&el=footer).
Last update
[5f06ae2...1f3142d](https://codecov.io/gh/apache/geode-native/pull/660?src=pr&el=lastupdated).
Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Reduce ThinClientLocatorHelper lock time
> ----------------------------------------
>
> Key: GEODE-8553
> URL: https://issues.apache.org/jira/browse/GEODE-8553
> Project: Geode
> Issue Type: Improvement
> Components: native client
> Affects Versions: 1.12.0, 1.13.0
> Reporter: Mario Salazar de Torres
> Assignee: Mario Salazar de Torres
> Priority: Major
> Labels: pull-request-available
>
> During my troublshootings, I've seen that locking m_locatorLock for the whole
> scope of the class function members might cause some inter-locks.
> Problem here and in many other places of the NC is that networking operations
> are performed while a mutex is locked. Therefore if *thread A* takes longer
> than expected in performing its network operation, it might block another one
> which does not requires any resource of the first *thread A*. Hence, the
> inter-lock.
> This improvement is the first one of a series regarding to lock scope
> reduction when it comes with code regarding networking in NC.
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
