{"id":38526,"date":"2025-04-24T04:22:08","date_gmt":"2025-04-23T20:22:08","guid":{"rendered":"https:\/\/www.yunshan.net\/blog\/?p=38526"},"modified":"2025-04-24T04:22:08","modified_gmt":"2025-04-23T20:22:08","slug":"skywalking%e5%a6%82%e4%bd%95%e5%b8%ae%e5%8a%a9%e5%9b%a2%e9%98%9f%e8%bf%9b%e8%a1%8c%e6%9c%89%e6%95%88%e7%9a%84%e4%bb%a3%e7%a0%81%e5%ae%a1%e6%9f%a5%e5%92%8c%e8%b4%a8%e9%87%8f%e4%bf%9d%e9%9a%9c","status":"publish","type":"post","link":"https:\/\/www.yunshan.net\/blog\/archives\/38526.html","title":{"rendered":"SkyWalking\u5982\u4f55\u5e2e\u52a9\u56e2\u961f\u8fdb\u884c\u6709\u6548\u7684\u4ee3\u7801\u5ba1\u67e5\u548c\u8d28\u91cf\u4fdd\u969c"},"content":{"rendered":"

\n

\"SkyWalking\u5982\u4f55\u5e2e\u52a9\u56e2\u961f\u8fdb\u884c\u6709\u6548\u7684\u4ee3\u7801\u5ba1\u67e5\u548c\u8d28\u91cf\u4fdd\u969c\"<\/p>\n<\/p>\n

\u4ee3\u7801\u5ba1\u67e5<\/h2>\n

1. \u89c4\u8303\u5ba1\u67e5\u6d41\u7a0b<\/strong><\/p>\n

  • SkyWalking\u793e\u533a\u4e2d\uff0c\u6bcf\u4e2aPull Request\uff08PR\uff09\u90fd\u9700\u8981\u7ecf\u8fc7\u6838\u5fc3\u7ef4\u62a4\u8005\u7684\u5ba1\u67e5\u624d\u80fd\u5408\u5e76\u3002\u89c4\u8303\u7684\u5ba1\u67e5\u6d41\u7a0b\u6709\u52a9\u4e8e\u53d1\u73b0\u6f5c\u5728\u7f3a\u9677\u3001\u4fdd\u6301\u4ee3\u7801\u98ce\u683c\u7edf\u4e00\u3001\u4f20\u64ad\u9879\u76ee\u77e5\u8bc6\u548c\u57f9\u517b\u5f00\u53d1\u8005\u534f\u4f5c\u80fd\u529b\u3002<\/li>\n

    2. \u5ba1\u67e5\u6838\u5fc3\u539f\u5219<\/strong><\/p>\n

  • \u57fa\u7840\u68c0\u67e5\u9879<\/strong>\uff1a<\/li>\n
  • PR\u6807\u9898\u7b26\u5408`[\u6a21\u5757] \u7b80\u77ed\u63cf\u8ff0`\u683c\u5f0f\uff08\u5982`[OAP] Fix metrics calculation bug`\uff09\u3002<\/li>\n
  • \u5173\u8054\u4e86\u5bf9\u5e94\u7684GitHub Issue\u3002<\/li>\n
  • \u6ca1\u6709\u5305\u542b\u65e0\u5173\u6587\u4ef6\u53d8\u66f4\u3002<\/li>\n
  • \u65b0\u589e\u4ee3\u7801\u8986\u76d6\u7387\u4e0d\u4f4e\u4e8e80%\u3002<\/li>\n
  • \u4ee3\u7801\u98ce\u683c\u89c4\u8303<\/strong>\uff1a<\/li>\n
  • Java\u4ee3\u7801\u4f7f\u7528Google Java Style\uff08\u901a\u8fc7`.\/mvnw checkstyle:check`\u9a8c\u8bc1\uff09\u3002<\/li>\n
  • \u524d\u7aef\u4ee3\u7801\u4f7f\u7528Prettier\u683c\u5f0f\u5316\u3002<\/li>\n
  • \u65e5\u5fd7\u5fc5\u987b\u4f7f\u7528\u9879\u76ee\u7ea6\u5b9a\u7684Logger\u5bf9\u8c61\u3002<\/li>\n
  • \u6d4b\u8bd5\u8981\u6c42<\/strong>\uff1a<\/li>\n
  • \u6240\u6709\u529f\u80fd\u4fee\u6539\u5fc5\u987b\u5305\u542b\u5355\u5143\u6d4b\u8bd5\uff08JUnit\/TestNG\uff09\u3002<\/li>\n
  • \u6d89\u53ca\u6a21\u5757\u95f4\u4ea4\u4e92\u65f6\u9700\u8981\u96c6\u6210\u6d4b\u8bd5\u3002<\/li>\n
  • \u82e5\u5f71\u54cd\u914d\u7f6e\u6216API\uff0c\u5219\u9700\u8981\u66f4\u65b0\u540e\u7684\u6587\u6863\u6d4b\u8bd5\u3002<\/li>\n

    3. \u5ba1\u67e5\u6d41\u7a0b\u8be6\u89e3<\/strong><\/p>\n

  • \u4ee5\u67d0\u5f00\u53d1\u8005\u63d0\u4ea4\u4e86OAP\u96c6\u7fa4\u901a\u4fe1\u7684\u4f18\u5316PR\u4e3a\u4f8b\uff0c\u5ba1\u67e5\u8981\u70b9\u5305\u62ec\u7ebf\u7a0b\u5b89\u5168\uff08\u5982\u6dfb\u52a0\u4e86`synchronized`\uff09\u3001\u6027\u80fd\u4f18\u5316\uff08\u5982\u6539\u4e3a\u589e\u91cf\u66f4\u65b0\uff09\uff0c\u5e76\u4e14\u9700\u8981\u8865\u5145\u5e76\u53d1\u6d4b\u8bd5\u7528\u4f8b\u3002<\/li>\n

    4. \u5e38\u89c1\u5ba1\u67e5\u95ee\u9898<\/strong><\/p>\n

  • \u9ad8\u9891\u95ee\u9898<\/strong>\uff1a<\/li>\n
  • \u8d44\u6e90\u6cc4\u6f0f\uff1a\u672a\u5173\u95ed\u7684\u6570\u636e\u5e93\u8fde\u63a5\u3001\u6587\u4ef6\u6d41\u7b49\u3002<\/li>\n
  • \u9b54\u6cd5\u6570\u5b57\uff1a\u672a\u5b9a\u4e49\u7684\u5e38\u91cf\u503c\uff08\u5982`if (status == 3)`\uff09\u3002<\/li>\n
  • \u8fc7\u5ea6\u8bbe\u8ba1\uff1a\u5f15\u5165\u4e0d\u5fc5\u8981\u7684\u590d\u6742\u6a21\u5f0f\u3002<\/li>\n
  • \u6d4b\u8bd5\u4e0d\u8db3\uff1a\u8fb9\u754c\u6761\u4ef6\u672a\u8986\u76d6\u3002<\/li>\n

    5. \u5982\u4f55\u53c2\u4e0e\u5ba1\u67e5<\/strong><\/p>\n

  • \u521d\u5b66\u8005\u53ef\u4ee5\u4ece`good first issue`\u6807\u8bb0\u7684PR\u5f00\u59cb\u5b66\u4e60\u3002\u4f7f\u7528`\/approve`\u6216`\/request-change`\u8bc4\u8bba\u65f6\uff0c\u9700\u5173\u6ce8\u6027\u80fd\u4f18\u5316\u662f\u5426\u5408\u7406\u3001\u6d4b\u8bd5\u8986\u76d6\u662f\u5426\u5b8c\u6574\u3001\u6587\u6863\u66f4\u65b0\u662f\u5426\u53ca\u65f6\u7b49\u3002\u5173\u6ce8\u6838\u5fc3\u7ef4\u62a4\u8005\uff08\u5982@wu\n
  • sheng\uff0c@kezhenxu94\uff09\u7684\u5ba1\u67e5\u610f\u89c1\u3002<\/li>\n<\/li>\n

    \u8d28\u91cf\u4fdd\u969c<\/h2>\n

    1. \u6027\u80fd\u76d1\u63a7\u4e0e\u5206\u6790<\/strong><\/p>\n

  • SkyWalking\u662f\u4e00\u6b3e\u5f00\u6e90\u7684\u5e94\u7528\u7a0b\u5e8f\u6027\u80fd\u76d1\u63a7\u5de5\u5177\uff0c\u80fd\u591f\u5e2e\u52a9\u5f00\u53d1\u4eba\u5458\u548cDevOps\u56e2\u961f\u76d1\u63a7\u5206\u5e03\u5f0f\u5e94\u7528\u7a0b\u5e8f\u7684\u6027\u80fd\uff0c\u4e86\u89e3\u5e94\u7528\u7a0b\u5e8f\u7684\u8fd0\u884c\u60c5\u51b5\uff0c\u4ece\u800c\u4fdd\u969c\u5e94\u7528\u7684\u6027\u80fd\u7b26\u5408\u7528\u6237\u7684\u671f\u671b\u548c\u4e1a\u52a1\u8981\u6c42\u3002<\/li>\n

    2. \u5206\u5e03\u5f0f\u94fe\u8def\u8ffd\u8e2a<\/strong><\/p>\n

  • SkyWalking\u63d0\u4f9b\u5206\u5e03\u5f0f\u94fe\u8def\u8ffd\u8e2a\u529f\u80fd\uff0c\u53ef\u4ee5\u5e2e\u52a9\u56e2\u961f\u770b\u5230\u4e00\u4e2a\u8bf7\u6c42\u7ecf\u8fc7\u4e86\u591a\u5c11\u4e2a\u5fae\u670d\u52a1\uff0c\u4e2d\u9014\u8c03\u7528\u4e86\u591a\u5c11\u6570\u636e\u5e93\u3001Redis\u3001MQ\u7b49\u4e2d\u95f4\u4ef6\uff0c\u6709\u52a9\u4e8e\u5b9a\u4f4d\u7cfb\u7edf\u6027\u80fd\u77ed\u677f\uff0c\u7f13\u89e3\u7cfb\u7edf\u538b\u529b\u3002<\/li>\n

    3. \u81ea\u52a8\u6d4b\u8bd5<\/strong><\/p>\n

  • \u4ece6.3.0\u7248\u672c\u5f00\u59cb\uff0cSkyWalking\u5f15\u5165\u4e86\u66f4\u591a\u7684\u81ea\u52a8\u6d4b\u8bd5\u6765\u6267\u884c\u8f6f\u4ef6\u8d28\u91cf\u4fdd\u8bc1\uff0cE2E\uff08\u7aef\u5230\u7aef\u6d4b\u8bd5\uff09\u662f\u5176\u4e2d\u6700\u91cd\u8981\u7684\u90e8\u5206\u4e4b\u4e00\uff0c\u7528\u4e8e\u6d4b\u8bd5\u5e94\u7528\u7a0b\u5e8f\u6d41\u7a0b\u4ece\u5934\u5230\u5c3e\u662f\u5426\u6309\u8bbe\u8ba1\u6267\u884c\u3002<\/li>\n

    4. \u6570\u636e\u53ef\u89c6\u5316<\/strong><\/p>\n

    SkyWalking\u63d0\u4f9b\u4e86\u591a\u79cd\u6570\u636e\u53ef\u89c6\u5316\u7684\u65b9\u5f0f\uff0c\u6709\u52a9\u4e8e\u8d28\u91cf\u4fdd\u969c\u56e2\u961f\u5168\u9762\u76d1\u63a7\u548c\u5206\u6790\u5e94\u7528\u7a0b\u5e8f\u7684\u6027\u80fd\u3002<\/p>\n","protected":false},"excerpt":{"rendered":"

    \u4ee3\u7801\u5ba1\u67e5 1. \u89c4\u8303\u5ba1\u67e5\u6d41\u7a0b SkyWalking\u793e\u533a\u4e2d\uff0c\u6bcf\u4e2aPull Request\uff08PR\uff09\u90fd\u9700\u8981\u7ecf\u8fc7\u6838\u5fc3\u7ef4\u62a4\u8005\u7684\u5ba1\u67e5\u624d\u80fd\u5408\u5e76\u3002\u89c4\u8303\u7684\u5ba1\u67e5\u6d41\u7a0b\u6709\u52a9\u4e8e\u53d1\u73b0\u6f5c\u5728\u7f3a\u9677\u3001\u4fdd\u6301\u4ee3\u7801\u98ce\u683c\u7edf\u4e00\u3001\u4f20\u64ad\u9879\u76ee\u77e5\u8bc6\u548c\u57f9\u517b\u5f00\u53d1\u8005\u534f\u4f5c\u80fd\u529b\u3002 2. \u5ba1\u67e5\u6838\u5fc3\u539f\u5219 \u57fa\u7840\u68c0\u67e5\u9879\uff1a PR\u6807\u9898\u7b26\u5408`[\u6a21\u5757] \u7b80\u77ed\u63cf\u8ff0`\u683c\u5f0f\uff08\u5982`[OAP] Fix metrics calculation bug`\uff09\u3002 \u5173\u8054\u4e86\u5bf9\u5e94\u7684GitHub Issue\u3002 \u6ca1\u6709\u5305\u542b\u65e0\u5173\u6587\u4ef6\u53d8\u66f4\u3002 \u65b0\u589e\u4ee3\u7801\u8986\u76d6\u7387\u4e0d\u4f4e\u4e8e80%\u3002 \u4ee3\u7801\u98ce\u683c\u89c4\u8303\uff1a Java\u4ee3\u7801\u4f7f\u7528Google Java Style\uff08\u901a\u8fc7`.\/mvnw checkstyle:check`\u9a8c\u8bc1\uff09\u3002 \u524d\u7aef\u4ee3\u7801\u4f7f\u7528Prettier\u683c\u5f0f\u5316\u3002 \u65e5\u5fd7\u5fc5\u987b\u4f7f\u7528\u9879\u76ee\u7ea6\u5b9a\u7684Logger\u5bf9\u8c61\u3002 \u6d4b\u8bd5\u8981\u6c42\uff1a \u6240\u6709\u529f\u80fd\u4fee\u6539\u5fc5\u987b\u5305\u542b\u5355\u5143\u6d4b\u8bd5\uff08JUnit\/TestNG\uff09\u3002 \u6d89\u53ca\u6a21\u5757\u95f4\u4ea4\u4e92\u65f6\u9700\u8981\u96c6\u6210\u6d4b\u8bd5\u3002 \u82e5\u5f71\u54cd\u914d\u7f6e\u6216API\uff0c\u5219\u9700\u8981\u66f4\u65b0\u540e\u7684\u6587\u6863\u6d4b\u8bd5\u3002 3. \u5ba1\u67e5\u6d41\u7a0b\u8be6\u89e3 \u4ee5\u67d0\u5f00\u53d1\u8005\u63d0\u4ea4\u4e86OAP\u96c6\u7fa4\u901a\u4fe1\u7684\u4f18\u5316PR\u4e3a\u4f8b\uff0c\u5ba1\u67e5\u8981\u70b9\u5305\u62ec\u7ebf\u7a0b\u5b89\u5168\uff08\u5982\u6dfb\u52a0\u4e86`synchronized`\uff09\u3001\u6027\u80fd\u4f18\u5316\uff08\u5982\u6539\u4e3a\u589e\u91cf\u66f4\u65b0\uff09\uff0c\u5e76\u4e14\u9700\u8981\u8865\u5145\u5e76\u53d1\u6d4b\u8bd5\u7528\u4f8b\u3002 4. \u5e38\u89c1\u5ba1\u67e5\u95ee\u9898 \u9ad8\u9891\u95ee\u9898\uff1a \u8d44\u6e90\u6cc4\u6f0f\uff1a\u672a\u5173\u95ed\u7684\u6570\u636e\u5e93\u8fde\u63a5\u3001\u6587\u4ef6\u6d41\u7b49\u3002 \u9b54\u6cd5\u6570\u5b57\uff1a\u672a\u5b9a\u4e49\u7684\u5e38\u91cf\u503c\uff08\u5982`if (status == 3)`\uff09\u3002 \u8fc7\u5ea6\u8bbe\u8ba1\uff1a\u5f15\u5165\u4e0d\u5fc5\u8981\u7684\u590d\u6742\u6a21\u5f0f\u3002 \u6d4b\u8bd5\u4e0d\u8db3\uff1a\u8fb9\u754c\u6761\u4ef6\u672a\u8986\u76d6\u3002 5. \u5982\u4f55\u53c2\u4e0e\u5ba1\u67e5 \u521d\u5b66\u8005\u53ef\u4ee5\u4ece`good first issue`\u6807\u8bb0\u7684PR\u5f00\u59cb\u5b66\u4e60\u3002\u4f7f\u7528`\/approve`\u6216`\/request-change`\u8bc4\u8bba\u65f6\uff0c\u9700\u5173\u6ce8\u6027\u80fd\u4f18\u5316\u662f\u5426\u5408\u7406\u3001\u6d4b\u8bd5\u8986\u76d6\u662f\u5426\u5b8c\u6574\u3001\u6587\u6863\u66f4\u65b0\u662f\u5426\u53ca\u65f6\u7b49\u3002\u5173\u6ce8\u6838\u5fc3\u7ef4\u62a4\u8005\uff08\u5982@wu sheng\uff0c@kezhenxu94\uff09\u7684\u5ba1\u67e5\u610f\u89c1\u3002 \u8d28\u91cf\u4fdd\u969c 1. \u6027\u80fd\u76d1\u63a7\u4e0e\u5206\u6790 SkyWalking\u662f\u4e00\u6b3e\u5f00\u6e90\u7684\u5e94\u7528\u7a0b\u5e8f\u6027\u80fd\u76d1\u63a7\u5de5\u5177\uff0c\u80fd\u591f\u5e2e\u52a9\u5f00\u53d1\u4eba\u5458\u548cDevOps\u56e2\u961f\u76d1\u63a7\u5206\u5e03\u5f0f\u5e94\u7528\u7a0b\u5e8f\u7684\u6027\u80fd\uff0c\u4e86\u89e3\u5e94\u7528\u7a0b\u5e8f\u7684\u8fd0\u884c\u60c5\u51b5\uff0c\u4ece\u800c\u4fdd\u969c\u5e94\u7528\u7684\u6027\u80fd\u7b26\u5408\u7528\u6237\u7684\u671f\u671b\u548c\u4e1a\u52a1\u8981\u6c42\u3002 2. \u5206\u5e03\u5f0f\u94fe\u8def\u8ffd\u8e2a SkyWalking\u63d0\u4f9b\u5206\u5e03\u5f0f\u94fe\u8def\u8ffd\u8e2a\u529f\u80fd\uff0c\u53ef\u4ee5\u5e2e\u52a9\u56e2\u961f\u770b\u5230\u4e00\u4e2a\u8bf7\u6c42\u7ecf\u8fc7\u4e86\u591a\u5c11\u4e2a\u5fae\u670d\u52a1\uff0c\u4e2d\u9014\u8c03\u7528\u4e86\u591a\u5c11\u6570\u636e\u5e93\u3001Redis\u3001MQ\u7b49\u4e2d\u95f4\u4ef6\uff0c\u6709\u52a9\u4e8e\u5b9a\u4f4d\u7cfb\u7edf\u6027\u80fd\u77ed\u677f\uff0c\u7f13\u89e3\u7cfb\u7edf\u538b\u529b\u3002 […]<\/p>\n","protected":false},"author":5,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[4],"tags":[],"class_list":["post-38526","post","type-post","status-publish","format-standard","hentry","category-discussion"],"_links":{"self":[{"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/posts\/38526","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/users\/5"}],"replies":[{"embeddable":true,"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/comments?post=38526"}],"version-history":[{"count":1,"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/posts\/38526\/revisions"}],"predecessor-version":[{"id":38527,"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/posts\/38526\/revisions\/38527"}],"wp:attachment":[{"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/media?parent=38526"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/categories?post=38526"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/www.yunshan.net\/blog\/wp-json\/wp\/v2\/tags?post=38526"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}