starrocks icon indicating copy to clipboard operation
starrocks copied to clipboard

[Feature] Implement HiveConnectorInternalMgr

Open stephen-shelby opened this issue 3 years ago • 2 comments
trafficstars

What type of PR is this:

  • [ ] BugFix
  • [x] Feature
  • [ ] Enhancement
  • [ ] Refactor
  • [ ] UT
  • [ ] Doc
  • [ ] Tool

Which issues of this PR fixes :

Fixes # https://github.com/StarRocks/starrocks/issues/11349

Problem Summary(Required) :

  • HiveConnectorInternalMgr is used to control life cycle of catalog-level IHiveMetastore and RemoteFiloIO's instance.

Checklist:

  • [ ] I have added test cases for my bug fix or my new feature
  • [ ] I have added user document for my new feature or new function

stephen-shelby avatar Sep 23 '22 12:09 stephen-shelby

[FE PR Coverage Check]

:disappointed: fail : 204 / 291 (70.10%)

file detail

path covered_line new_line coverage not_covered_line_detail
:large_blue_circle: com/starrocks/connector/hive/HiveConnectorInternalMgr.java 0 43 00.00% [38, 39, 40, 41, 42, 43, 46, 47, 48, 50, 53, 54, 56, 57, 59, 62, 63, 66, 67, 69, 70, 71, 74, 75, 76, 78, 82, 83, 85, 86, 88, 89, 90, 93, 94, 95, 96, 99, 103, 104, 105, 106, 107]
:large_blue_circle: com/starrocks/external/hive/HiveStatisticsProvider.java 202 246 82.11% [68, 69, 117, 118, 172, 173, 182, 183, 184, 186, 187, 188, 190, 225, 250, 251, 252, 253, 255, 293, 323, 337, 349, 350, 351, 352, 353, 355, 356, 359, 360, 361, 362, 364, 366, 417, 420, 422, 427, 430, 432, 440, 443, 465]
:large_blue_circle: com/starrocks/qe/SessionVariable.java 2 2 100.00% []

wanpengfei-git avatar Sep 23 '22 12:09 wanpengfei-git