{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D5","name":"Coupling","shortDescription":{"text":"Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D5"},{"id":"D13","name":"Secret Scanning","shortDescription":{"text":"Secret Scanning"},"helpUri":"https://codehealth.canine.dev/dimensions/D13","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D14","name":"License Compliance","shortDescription":{"text":"License Compliance"},"helpUri":"https://codehealth.canine.dev/dimensions/D14"},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"id":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D17","name":"Explicit Debt","shortDescription":{"text":"Explicit Debt"},"helpUri":"https://codehealth.canine.dev/dimensions/D17"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D26","name":"Project Cohesion","shortDescription":{"text":"Project Cohesion"},"helpUri":"https://codehealth.canine.dev/dimensions/D26"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D30","name":"Dependency Vulnerabilities","shortDescription":{"text":"Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D30","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"DefaultStorage.pollFromConsumerLog (cyclomatic 21): DefaultStorage.pollFromConsumerLog has cyclomatic complexity 21 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/DefaultStorage.java"},"region":{"startLine":258}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a4693593afe8553a1cfb30a47649c6e8797bc7dc07bf2515fa7e5fc1039fb7e"}},{"ruleId":"D1","level":"warning","message":{"text":"MessageStoreWrapper.doFindUnAckMessages (cyclomatic 20): MessageStoreWrapper.doFindUnAckMessages has cyclomatic complexity 20 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-server/src/main/java/qunar/tc/qmq/store/MessageStoreWrapper.java"},"region":{"startLine":279}}}],"partialFingerprints":{"codehealthFindingId/v1":"4cb5748b3024a5375131e9db1111815b68e6dd7b048327914eae451efba894bb"}},{"ruleId":"D1","level":"warning","message":{"text":"DelayMessageLogVisitor.readOneRecord (cyclomatic 16): DelayMessageLogVisitor.readOneRecord has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"1eeb710b9dac1022352471da0ca3aa5eedc09c815bf0bd1b0acb1799ed199b77"}},{"ruleId":"D2","level":"warning","message":{"text":"DefaultStorage.pollFromConsumerLog (cognitive 40): DefaultStorage.pollFromConsumerLog has cognitive complexity 40 (threshold 15). Drivers by points: if/else 37, boolean chains 1, loops 1, ternaries 1 (nesting depth added 19). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/DefaultStorage.java"},"region":{"startLine":258}}}],"partialFingerprints":{"codehealthFindingId/v1":"aba52628d6a754f98c76c71dd63e5c0ccfff564829ffb3077e69463c6531790b"}},{"ruleId":"D2","level":"warning","message":{"text":"DelayMessageLogVisitor.readOneRecord (cognitive 38): DelayMessageLogVisitor.readOneRecord has cognitive complexity 38 (threshold 15). Drivers by points: if/else 38 (nesting depth added 22). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"1fabfeadf2e79d23c6c38077beb5111134c4aaa6a121ae20df38093fd8181cc9"}},{"ruleId":"D2","level":"warning","message":{"text":"MessageStoreWrapper.doFindUnAckMessages (cognitive 38): MessageStoreWrapper.doFindUnAckMessages has cognitive complexity 38 (threshold 15). Drivers by points: if/else 32, boolean chains 3, error handling 2, loops 1 (nesting depth added 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-server/src/main/java/qunar/tc/qmq/store/MessageStoreWrapper.java"},"region":{"startLine":279}}}],"partialFingerprints":{"codehealthFindingId/v1":"4d40d1a771e47360961072965674953e2921da8fe1a83d457c5136bdc6b0b217"}},{"ruleId":"D2","level":"warning","message":{"text":"SocketChannel.Flush (cognitive 28): SocketChannel.Flush has cognitive complexity 28 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/SocketChannel.cs"},"region":{"startLine":146}}}],"partialFingerprints":{"codehealthFindingId/v1":"b70488ac4e28b76a39702391e811e25655486051b8f4dc1d010afcc53478be00"}},{"ruleId":"D2","level":"warning","message":{"text":"MessageLogRecordVisitor.readOneRecord (cognitive 26): MessageLogRecordVisitor.readOneRecord has cognitive complexity 26 (threshold 15). Drivers by points: if/else 26 (nesting depth added 14). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/MessageLogRecordVisitor.java"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"ad772c96a272c4fb445fbd3325a9f6fec0549b128f21ddc6746a33ea3e63e8d3"}},{"ruleId":"D2","level":"warning","message":{"text":"MinBinaryHeap.Remove (cognitive 24): MinBinaryHeap.Remove has cognitive complexity 24 (threshold 15). To reduce it, flatten the nesting: invert conditions into early returns or guard clauses so the happy path stays at one level, and lift the deepest nested block into its own named function."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Util/ConcurrentPriorityQueue.cs"},"region":{"startLine":288}}}],"partialFingerprints":{"codehealthFindingId/v1":"d2074b7a10b00c658cfc2d57cdc427402fedb4489bd2d93574b76b5fbeecbb56"}},{"ruleId":"D2","level":"warning","message":{"text":"NetworkUtils.getLocalAddress (cognitive 24): NetworkUtils.getLocalAddress has cognitive complexity 24 (threshold 15). Drivers by points: if/else 18, loops 5, error handling 1 (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-common/src/main/java/qunar/tc/qmq/utils/NetworkUtils.java"},"region":{"startLine":47}}}],"partialFingerprints":{"codehealthFindingId/v1":"bcce04ebb0c4a0fbd50c889e796b69822cac09d6cd333371fedd2112b2231b04"}},{"ruleId":"D2","level":"warning","message":{"text":"IndexLog.copy (cognitive 23): IndexLog.copy has cognitive complexity 23 (threshold 15). Drivers by points: if/else 22, loops 1 (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/IndexLog.java"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"1eb5c5b3719dd4ce8d5fce13054d6882a704d29374452e21e57493fc6c57ecc6"}},{"ruleId":"D2","level":"warning","message":{"text":"MessageIndexSyncWorker.getSyncLog (cognitive 22): MessageIndexSyncWorker.getSyncLog has cognitive complexity 22 (threshold 15). Drivers by points: if/else 21, loops 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-server/src/main/java/qunar/tc/qmq/sync/master/MessageIndexSyncWorker.java"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"161a1d7520cdac11516830756ff6cf542116842878340672edf69fb6d3da0b87"}},{"ruleId":"D2","level":"warning","message":{"text":"WeightLoadBalance.select (cognitive 21): WeightLoadBalance.select has cognitive complexity 21 (threshold 15). Drivers by points: if/else 15, boolean chains 3, loops 3 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-client/src/main/java/qunar/tc/qmq/consumer/pull/WeightLoadBalance.java"},"region":{"startLine":45}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d71636ca2dc12bccffb2b0417bd54721ddcfac539d54bb4ccd1daf4836b9580"}},{"ruleId":"D2","level":"warning","message":{"text":"SenderGroup.send (cognitive 21): SenderGroup.send has cognitive complexity 21 (threshold 15). Drivers by points: if/else 15, boolean chains 3, loops 2, error handling 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/sender/SenderGroup.java"},"region":{"startLine":120}}}],"partialFingerprints":{"codehealthFindingId/v1":"6b531ce4ce9926b0c244f175394db0bf47c6df435ec0f788c2cec9ef6ed4219e"}},{"ruleId":"D2","level":"warning","message":{"text":"ActionSyncLogIterator.next (cognitive 20): ActionSyncLogIterator.next has cognitive complexity 20 (threshold 15). Drivers by points: if/else 20 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/ActionSyncLogIterator.java"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"4f40662f9cdd03e422b7f0ae43bf062eb48605c35650657e626bc9d8579bfbab"}},{"ruleId":"D2","level":"warning","message":{"text":"SendServlet.appendAttrs (cognitive 20): SendServlet.appendAttrs has cognitive complexity 20 (threshold 15). Drivers by points: if/else 18, boolean chains 1, loops 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-gateway/src/main/java/qunar/tc/qmq/gateway/servlet/SendServlet.java"},"region":{"startLine":103}}}],"partialFingerprints":{"codehealthFindingId/v1":"84bcb5707b02c561440c896eba9fa486f1b27b41f96557c201c436bdd03096aa"}},{"ruleId":"D2","level":"warning","message":{"text":"ActionLogVisitor.readOneRecord (cognitive 20): ActionLogVisitor.readOneRecord has cognitive complexity 20 (threshold 15). Drivers by points: if/else 16, error handling 4 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/ActionLogVisitor.java"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"a75d7a73924784a9858cfd352e1c22d2630fc1b2dff1699bd15ef9495bd45417"}},{"ruleId":"D2","level":"warning","message":{"text":"DefaultPullEntry.run (cognitive 19): DefaultPullEntry.run has cognitive complexity 19 (threshold 15). Drivers by points: if/else 13, error handling 5, match/switch 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-client/src/main/java/qunar/tc/qmq/consumer/pull/DefaultPullEntry.java"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"487d3b34094e114d3e05e9c4da2220a2481b48c0eaa08a0cc8cfaed71542ff54"}},{"ruleId":"D2","level":"warning","message":{"text":"NettyConnectManageHandler.getOrCreateChannel (cognitive 19): NettyConnectManageHandler.getOrCreateChannel has cognitive complexity 19 (threshold 15). Drivers by points: if/else 17, boolean chains 1, error handling 1 (nesting depth added 4). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-remoting/src/main/java/qunar/tc/qmq/netty/client/NettyConnectManageHandler.java"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"299e35a422bd778d5f835cdac316c51ae2da924903e6c385280cad526f8423ea"}},{"ruleId":"D2","level":"warning","message":{"text":"SendMessageTask.processErrorMessages (cognitive 19): SendMessageTask.processErrorMessages has cognitive complexity 19 (threshold 15). Drivers by points: if/else 10, error handling 5, ternaries 3, loops 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-watchdog/src/main/java/qunar/tc/qmq/task/SendMessageTask.java"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"00f75b8c27258e8ba9c74bb52091d17c62f73e0ca5fb1a63db3cd3706a243d9d"}},{"ruleId":"D2","level":"warning","message":{"text":"PollBrokerLoadBalance.loadBalance (cognitive 18): PollBrokerLoadBalance.loadBalance has cognitive complexity 18 (threshold 15). Drivers by points: if/else 11, loops 6, boolean chains 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-common/src/main/java/qunar/tc/qmq/broker/impl/PollBrokerLoadBalance.java"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"48f5516597d70c3b3f1b43c77636f8fe72e28538c92bceb9cec80702fa81afb6"}},{"ruleId":"D2","level":"warning","message":{"text":"WeightLoadBalance.Select (cognitive 17): WeightLoadBalance.Select has cognitive complexity 17 (threshold 15). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/NewQmq/WeightLoadBalance.cs"},"region":{"startLine":18}}}],"partialFingerprints":{"codehealthFindingId/v1":"6cf7fee4963e8fadd0a63719f9ced35e864fa9ad2699c897497e6964680925ce"}},{"ruleId":"D2","level":"warning","message":{"text":"DefaultPullConsumer.AsyncPull (cognitive 17): DefaultPullConsumer.AsyncPull has cognitive complexity 17 (threshold 15). Most of this is not in the body itself: 3 of the 17 points are its own statements and the rest belongs to one function literal inside it that branches (line 141). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Pull/DefaultPullConsumer.cs"},"region":{"startLine":127}}}],"partialFingerprints":{"codehealthFindingId/v1":"58c66fd59ff44c773f0f48922cf6cdceece906c6795f80fd3157aff6f2637b5e"}},{"ruleId":"D2","level":"warning","message":{"text":"HBaseRecordStore.findRecords (cognitive 17): HBaseRecordStore.findRecords has cognitive complexity 17 (threshold 15). Drivers by points: if/else 17 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HBaseRecordStore.java"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"f9135c74f51697d03b73baf989bd84b7a581d4d9ffc36769b69898640e29cfa6"}},{"ruleId":"D2","level":"warning","message":{"text":"1.onLoad (cognitive 17): 1.onLoad has cognitive complexity 17 (threshold 15). Drivers by points: if/else 11, loops 5, boolean chains 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-client/src/main/java/qunar/tc/qmq/config/PullSubjectsConfig.java"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"ae124cf78360593fb36e51a2a0bf22cc74c822d7a23171e5487bc0dc947e37c0"}},{"ruleId":"D2","level":"warning","message":{"text":"HBaseStore.scan (cognitive 16): HBaseStore.scan has cognitive complexity 16 (threshold 15). Drivers by points: if/else 12, boolean chains 2, loops 2 (nesting depth added 3). To reduce it, split the body: this score is breadth rather than depth \u2014 many checks laid out side by side rather than nested inside one another, so inverting conditions into early returns has nothing left to flatten. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HBaseStore.java"},"region":{"startLine":91}}}],"partialFingerprints":{"codehealthFindingId/v1":"6915bb5a21d3ef9a9f677224e9eb4772d63b58217c62cef6179b399f1964feaa"}},{"ruleId":"D2","level":"warning","message":{"text":"MetaInfoService.parseResponse (cognitive 16): MetaInfoService.parseResponse has cognitive complexity 16 (threshold 15). Drivers by points: if/else 10, boolean chains 3, loops 2, ternaries 1 (nesting depth added 2). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-remoting/src/main/java/qunar/tc/qmq/metainfoclient/MetaInfoService.java"},"region":{"startLine":188}}}],"partialFingerprints":{"codehealthFindingId/v1":"6792db1c49d08e9a7bf03854ab44a30557433162e7e5e3ef4d92895bd7f5365e"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: QMon: TooManyMethods \u2014 89 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-server-common/src/main/java/qunar/tc/qmq/monitor/QMon.java"},"region":{"startLine":32}}}],"partialFingerprints":{"codehealthFindingId/v1":"6ab2aa9a5111ffa1ade2e53a6da91ae944e31b401e02b5c59eef787ee28da9a2"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: BaseMessage: TooManyMethods \u2014 62 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-common/src/main/java/qunar/tc/qmq/base/BaseMessage.java"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b4f6061b2a93ccd9cfb2625bfa8fd83efb64a1c89468a3667b2c08f77e3ee02"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: CheckpointManager: TooManyMethods \u2014 52 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/CheckpointManager.java"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"c42cb07f128cf0b5e47312e9181a7bc2a168ae066a8b91c2c27bfa5b61668ee3"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: DefaultStorage: TooManyMethods \u2014 42 methods. Most of these members implement store.Storage (31 of 40 members), so moving them onto a smaller type would remove them from that contract rather than reduce it. To reduce it, split the contract instead: give each cohesive group of operations its own smaller interface and its own implementing type. Where the contract has to stay whole, move the work behind these members into collaborator types, so what is left here is a forward per member rather than a responsibility per member."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/DefaultStorage.java"},"region":{"startLine":48}}}],"partialFingerprints":{"codehealthFindingId/v1":"6a69f4db546b4b153999b22eeec7130d49d4e1a2da75f22e9c70f72b6255d68f"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: BaseMessage: TooManyMethods \u2014 208 significant lines (blank, comment-only and punctuation-only lines excluded), 38 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Model/BaseMessage.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"f1b16d3303bb7059fa84807e29cc1861e01fcaa33611626ce7697aac680e5ab1"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: QMon: TooManyMethods \u2014 33 methods. To reduce it, group the members that share the same data into a smaller type of their own and delegate to it, so no single type carries every responsibility."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/monitor/QMon.java"},"region":{"startLine":29}}}],"partialFingerprints":{"codehealthFindingId/v1":"f26618f4346fedcfa31f32df8c4b98e790459d49205f830702985fb66f1bbd42"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:91-103 | qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:104-115 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:91\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java"},"region":{"startLine":91}}}],"partialFingerprints":{"codehealthFindingId/v1":"e60ab0b1ec23a710ba099a201e19e05764002a59121edd207ffce8ce0368e3a7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:112-124 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLogRecordVisitor.java:71-82 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:112\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java"},"region":{"startLine":112}}}],"partialFingerprints":{"codehealthFindingId/v1":"2240e42bf93472f19912eb77993445fd5ede579e579f5eb03714401d0c065381"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java:110-120 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageContentBatchBackup.java:102-112 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java:110\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java"},"region":{"startLine":110}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4f863f69dde1b650e872e4ac6f36984a210fc339c97a7da1b87956bd0ca13c3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:84-94 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:104-114 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:84\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java"},"region":{"startLine":84}}}],"partialFingerprints":{"codehealthFindingId/v1":"46e000ff4df9409af370a3402567c72118f46c80839395d8af629f6aa25a32c5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:134-144 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:162-172 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:134\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java"},"region":{"startLine":134}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d10a28af07523c2799064404873f9e18c4b18344cc8512b63a03ad96932d8cb"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/AbstractHBaseMessageStore.java:68-78 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HBaseDeadMessageContentStore.java:98-108 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/AbstractHBaseMessageStore.java:68\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/AbstractHBaseMessageStore.java"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"ad8ae70bd86f827cda59ba5f8b1fdb3cfe55393b3340331952e5fc94e2d6a3b5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/receiver/ReceivedDelayMessageProcessor.java:56-66 | qmq-server/src/main/java/qunar/tc/qmq/processor/SendMessageProcessor.java:72-82 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-server/src/main/java/qunar/tc/qmq/processor/SendMessageProcessor.java:72\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/receiver/ReceivedDelayMessageProcessor.java"},"region":{"startLine":56}}}],"partialFingerprints":{"codehealthFindingId/v1":"8d4e485f1e2a4d0b22b9bf6f12f95ea691b9b2327a357c4c90629841fa2f7d3e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/DispatchLogSegmentContainer.java:57-67 | qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/ScheduleSetSegmentContainer.java:52-62 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/DispatchLogSegmentContainer.java:57\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/DispatchLogSegmentContainer.java"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"fc7d46afaac7ae7591def2c621f904cbbef25ff52dd15a607d34a5ad6c705fd3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java:357-367 | qmq-store/src/main/java/qunar/tc/qmq/store/ActionLog.java:189-199 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java:357\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java"},"region":{"startLine":357}}}],"partialFingerprints":{"codehealthFindingId/v1":"4712ab46f00aff23565804e4d55c6d129961bd93d7417ec1a7788140f09dcff4"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-store/src/main/java/qunar/tc/qmq/store/ActionLog.java:53-63 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLog.java:82-92 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/ActionLog.java"},"region":{"startLine":53}}}],"partialFingerprints":{"codehealthFindingId/v1":"50190e561e012f69f3b0277dd4e2cdfcb0e4644ddf3230ef4bed694ed30d6ed7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): qmq-store/src/main/java/qunar/tc/qmq/store/IndexLogVisitor.java:63-73 | qmq-store/src/main/java/qunar/tc/qmq/store/IndexLogVisitor.java:76-86 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/IndexLogVisitor.java"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"c8742e3c135acc7114501764e53a16f6f1fd6e15fe9fdf01f5533f708fdfe8d5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/DispatchLogSegmentContainer.java:71-80 | qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/ScheduleSetSegmentContainer.java:66-75 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/DispatchLogSegmentContainer.java:71\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/DispatchLogSegmentContainer.java"},"region":{"startLine":71}}}],"partialFingerprints":{"codehealthFindingId/v1":"69233aa3a30561a6c76b77250037f760d02477647c2180b9949466ea14f5101f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): qmq-store/src/main/java/qunar/tc/qmq/store/DefaultStorage.java:623-632 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLog.java:169-178 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-store/src/main/java/qunar/tc/qmq/store/DefaultStorage.java:623\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/DefaultStorage.java"},"region":{"startLine":623}}}],"partialFingerprints":{"codehealthFindingId/v1":"89d4e56c708aec14baadcc41b6916f1270681e4b80a4dff1c0a4eb3df0ec1ec3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:158-166 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:168-176 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:158\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java"},"region":{"startLine":158}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea8f3c8cc10b108b4c36286456955ea6a868424f891f8b56d52ff4af04cc6dcb"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): qmq-metaserver/src/main/java/qunar/tc/qmq/meta/management/AddBrokerAction.java:89-97 | qmq-metaserver/src/main/java/qunar/tc/qmq/meta/management/ReplaceBrokerAction.java:84-92 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-metaserver/src/main/java/qunar/tc/qmq/meta/management/AddBrokerAction.java"},"region":{"startLine":89}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a85cd860a17eea78f7df5037089758d7fe7fe58001b673066a14056e809b1b3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:46-56 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLogRecordVisitor.java:37-45 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:46\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java"},"region":{"startLine":46}}}],"partialFingerprints":{"codehealthFindingId/v1":"50625a431b5bc4e831095de0b2391f03613dff0e832433e8ff05318eecdc51ac"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): qmq-store/src/main/java/qunar/tc/qmq/store/LogManager.java:65-73 | qmq-store/src/main/java/qunar/tc/qmq/store/SortedPullLogTable.java:215-223 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-store/src/main/java/qunar/tc/qmq/store/LogManager.java:65\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/LogManager.java"},"region":{"startLine":65}}}],"partialFingerprints":{"codehealthFindingId/v1":"518387fe73eca48c0c80018dea8a36a24caa2be6f72b2a98d2d18c5b19d803b9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:120-127 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:149-156 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:120\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java"},"region":{"startLine":120}}}],"partialFingerprints":{"codehealthFindingId/v1":"05c95d3cf6e139ccc2f793b3873e4fd61312d6703c949dfe5350a5a4933cdfd5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:177-184 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:202-209 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java:177\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageServiceImpl.java"},"region":{"startLine":177}}}],"partialFingerprints":{"codehealthFindingId/v1":"00b91d518f43047df1ed33577fac5e2a6a5d1e59be8ae3bbdf3946bc10530393"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:172-179 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:211-218 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:172\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java"},"region":{"startLine":172}}}],"partialFingerprints":{"codehealthFindingId/v1":"0f480233e549f053c54ed8bb98fcc110dd6286d429d6893ada4cb269858c188c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HBaseIndexStore.java:80-87 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HBaseStore.java:65-72 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HBaseIndexStore.java"},"region":{"startLine":80}}}],"partialFingerprints":{"codehealthFindingId/v1":"3d87a4fcdb94445c7775129d6f09c79fe6732cf84405b23a22b11da02a6b150f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:171-178 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:181-188 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:171\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java"},"region":{"startLine":171}}}],"partialFingerprints":{"codehealthFindingId/v1":"9982bfdecb9fed14d18f6b59799db73c2896be4fcba08c5d6be7bd4f96343b6f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/receiver/ReceivedDelayMessageProcessor.java:90-97 | qmq-server/src/main/java/qunar/tc/qmq/processor/SendMessageProcessor.java:103-110 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/receiver/ReceivedDelayMessageProcessor.java"},"region":{"startLine":90}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e46a6c0d793cfd731fe67a57ad467ca828e52c108df3fef09dc874236aec3d3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-gateway/src/main/java/qunar/tc/qmq/gateway/servlet/SendServlet.java:166-173 | qmq-gateway/src/main/java/qunar/tc/qmq/gateway/servlet/SendServlet.java:181-188 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-gateway/src/main/java/qunar/tc/qmq/gateway/servlet/SendServlet.java:166\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-gateway/src/main/java/qunar/tc/qmq/gateway/servlet/SendServlet.java"},"region":{"startLine":166}}}],"partialFingerprints":{"codehealthFindingId/v1":"be4039ef6d5671e6fb1cc94f24956d09fd81d72f340672f19baf48120a314b4e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): qmq-store/src/main/java/qunar/tc/qmq/store/ActionLogVisitor.java:58-65 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLogRecordVisitor.java:54-61 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-store/src/main/java/qunar/tc/qmq/store/ActionLogVisitor.java:58\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-store/src/main/java/qunar/tc/qmq/store/ActionLogVisitor.java"},"region":{"startLine":58}}}],"partialFingerprints":{"codehealthFindingId/v1":"b77e45b25043ae6b121e0affca150fe11cb632b4683642adccd579dea45013ec"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 3): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:66-73 | qmq-store/src/main/java/qunar/tc/qmq/store/ActionLogVisitor.java:57-63 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLogRecordVisitor.java:53-59 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 3 call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made 3 times. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java:66\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/visitor/DelayMessageLogVisitor.java"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"5700a05ea3162a90355fa2c366be6f4849a2132eeca91dc70b02d8c21b67ac5e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageIndexBatchBackup.java:69-75 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:112-118 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageIndexBatchBackup.java"},"region":{"startLine":69}}}],"partialFingerprints":{"codehealthFindingId/v1":"bf4044b42a99a1c1860aa172c12143a581481db133c80cc59d7d45951f753ae7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:113-119 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:116-122 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:113\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java"},"region":{"startLine":113}}}],"partialFingerprints":{"codehealthFindingId/v1":"ada10f9023fe3a5c45c37ad7e664f0b293af96b0541e9e5e8b033175a028c466"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:191-197 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:230-236 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:191\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java"},"region":{"startLine":191}}}],"partialFingerprints":{"codehealthFindingId/v1":"5af975fd8bd4da27d10f4d247c72c6dfb77526099083d40239a76487f4370c0f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:214-220 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:253-259 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:214\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java"},"region":{"startLine":214}}}],"partialFingerprints":{"codehealthFindingId/v1":"b58231b63026f0aca6bd146c71535d2bd89256e2cbf4e1d2e736d9d5547dba5e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): qmq-client/src/main/java/qunar/tc/qmq/producer/ProduceMessageImpl.java:149-155 | qmq-client/src/main/java/qunar/tc/qmq/producer/ProduceMessageImpl.java:193-199 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-client/src/main/java/qunar/tc/qmq/producer/ProduceMessageImpl.java:149\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-client/src/main/java/qunar/tc/qmq/producer/ProduceMessageImpl.java"},"region":{"startLine":149}}}],"partialFingerprints":{"codehealthFindingId/v1":"fde46893ba4ed94ff76f7643133321d279bc28603757f4bd59b84dcefeed9746"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java:311-317 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLog.java:263-269 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java"},"region":{"startLine":311}}}],"partialFingerprints":{"codehealthFindingId/v1":"4db46c7fede48a7291cba2cab5994930785b94185f185fae62834419711ed2c3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): qmq-tools/src/main/java/qunar/tc/qmq/tools/command/AddBrokerCommand.java:64-70 | qmq-tools/src/main/java/qunar/tc/qmq/tools/command/ReplaceBrokerCommand.java:65-71 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-tools/src/main/java/qunar/tc/qmq/tools/command/AddBrokerCommand.java"},"region":{"startLine":64}}}],"partialFingerprints":{"codehealthFindingId/v1":"cff8841bb97888d306b4140c63b1f51aaaefba4909b629d3beda323a771acd3b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java:84-89 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageContentBatchBackup.java:72-77 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java:84\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java"},"region":{"startLine":84}}}],"partialFingerprints":{"codehealthFindingId/v1":"bf8280e06f364096b5ce49bdf7745f8cdfba1f084487889c7710b81e6832fc8a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:123-128 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:126-131 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java"},"region":{"startLine":123}}}],"partialFingerprints":{"codehealthFindingId/v1":"dbd152e30fa9518779d1342770eb6237847594d33cd6168b035434207c5306b1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:82-87 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:89-94 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java"},"region":{"startLine":82}}}],"partialFingerprints":{"codehealthFindingId/v1":"b878fb7080df7d2bb9c3766c9e8995b62a33ea234c1c75a34469c8285593ee74"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): qmq-server/src/main/java/qunar/tc/qmq/utils/HeaderSerializer.java:36-45 | qmq-remoting/src/main/java/qunar/tc/qmq/netty/EncodeHandler.java:56-61 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-server/src/main/java/qunar/tc/qmq/utils/HeaderSerializer.java"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ff77c1135e167d1197b154c3aa32f9523ffb02197b6e77597c171cb935defe2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java:121-126 | qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java:399-404 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java:121\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"0a36f9322e7ef0ff1a61b49ec83738bb3e138f6d9a1700fa1ae5bb1d95545758"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): qmq-tools/src/main/java/qunar/tc/qmq/tools/command/AddSubjectBrokerGroupCommand.java:51-56 | qmq-tools/src/main/java/qunar/tc/qmq/tools/command/RemoveSubjectBrokerGroupCommand.java:51-56 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-tools/src/main/java/qunar/tc/qmq/tools/command/AddSubjectBrokerGroupCommand.java"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"af6bdf0e57220cb93f0314b60c72ce9fcc1d91a9407758c537c280520662cc40"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (6 lines \u00D7 2): qmq-tools/src/main/java/qunar/tc/qmq/tools/command/MarkBrokerReadonlyCommand.java:52-57 | qmq-tools/src/main/java/qunar/tc/qmq/tools/command/UnMarkBrokerReadonlyCommand.java:52-57 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-tools/src/main/java/qunar/tc/qmq/tools/command/MarkBrokerReadonlyCommand.java"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"632d5a1f988f1b72cf55e42e69d79bd9fe681bd8dc3f232645341b2660da63c7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 3): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java:100-104 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/MessageIndexBatchBackup.java:77-81 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:120-124 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere all 3 call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made 3 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java"},"region":{"startLine":100}}}],"partialFingerprints":{"codehealthFindingId/v1":"4379d0016fd1ceed463d293c959859f7ef7d6ed9090b6a0c1b92f224e3b0d4fd"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 3): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/BackupKeyGenerator.java:52-56 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/BackupKeyGenerator.java:64-68 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/BackupKeyGenerator.java:75-79 \u2014 all 3 copies are in the same file, so extract the block into one function there and call it from every one of those sites \u2014 resolving only two of them leaves the rest to drift apart the first time one is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/BackupKeyGenerator.java:52\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/BackupKeyGenerator.java"},"region":{"startLine":52}}}],"partialFingerprints":{"codehealthFindingId/v1":"3a28cf566c148f4692e40675773300582f23cc055beb557c713e5fa22ecc6b64"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java:95-99 | qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageContentBatchBackup.java:92-96 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/DeadMessageBatchBackup.java"},"region":{"startLine":95}}}],"partialFingerprints":{"codehealthFindingId/v1":"f21e6f2efc2f0a127381a5f9e9acca67a303d6097e1011c4f9dd5db0aae8f76a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:206-210 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:245-249 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java:206\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/service/impl/RecordBatchBackup.java"},"region":{"startLine":206}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d6007d6cd7d08da9afe0077a84ddfe286f1a80ccb6a661b4df05d554cae6bbb"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/AbstractHBaseMessageStore.java:61-65 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HBaseDeadMessageContentStore.java:86-90 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/AbstractHBaseMessageStore.java:61\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/AbstractHBaseMessageStore.java"},"region":{"startLine":61}}}],"partialFingerprints":{"codehealthFindingId/v1":"48caa40abcffac896c6fef0cd796492111b81dc3dae00e56f7f6636b15e8696b"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java:94-98 | qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileRecordStore.java:101-105 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-backup/src/main/java/qunar/tc/qmq/backup/store/impl/HFileIndexStore.java"},"region":{"startLine":94}}}],"partialFingerprints":{"codehealthFindingId/v1":"45ece77cc2c0b5ebd69099550363b216dcd5aeea405944007748844a0d656ef8"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): qmq-client/src/main/java/qunar/tc/qmq/consumer/pull/PullService.java:220-224 | qmq-remoting/src/main/java/qunar/tc/qmq/protocol/QMQSerializer.java:115-119 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060qmq-client/src/main/java/qunar/tc/qmq/consumer/pull/PullService.java:220\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-client/src/main/java/qunar/tc/qmq/consumer/pull/PullService.java"},"region":{"startLine":220}}}],"partialFingerprints":{"codehealthFindingId/v1":"76303cb7fa38462b4bf65524d4f719a896689be9db278ad9e7abeb115500679e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java:329-333 | qmq-store/src/main/java/qunar/tc/qmq/store/MessageLog.java:280-284 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"qmq-delay-server/src/main/java/qunar/tc/qmq/delay/store/log/MessageSegmentContainer.java"},"region":{"startLine":329}}}],"partialFingerprints":{"codehealthFindingId/v1":"798cfde1710f8c8a1b9939b63952d40e0b705fdd14606980b978be487c6f4ddd"}},{"ruleId":"D16","level":"warning","message":{"text":"dormant codebase \u2014 no living knowledge left to concentrate: Every one of the 36 significant source file(s) was last meaningfully changed so long ago that no living knowledge remains, so there is no concentration to measure \u2014 the bus factor is not scored. This is not a clean bill: nobody currently holds working knowledge of this code (see D34 Knowledge Freshness)."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"569d55ea5cb0330f13c125b289d07e5abdfcceac947a2c2645277937942e7d5f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(keli.wang): make timeout configable \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Cluster/NewQmqCluster.cs"},"region":{"startLine":66}}}],"partialFingerprints":{"codehealthFindingId/v1":"83f7b7dbf240db269e4a18aa716e3e7f91f5b9e0073deb0e72ae4232738fef15"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Metainfo/MetainfoAddressResolver.cs"},"region":{"startLine":59}}}],"partialFingerprints":{"codehealthFindingId/v1":"ae6f4f46e564160683b4cf264819e03dbafa11bafc7eca215df6c7390770c672"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/NewQmq/NewQmqClient.cs"},"region":{"startLine":33}}}],"partialFingerprints":{"codehealthFindingId/v1":"a7cb41209d77668f08b902c72b35f1fb6ad51acee4b8b84c7c182c1385b25b2a"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/ProducerMessageImpl.cs"},"region":{"startLine":131}}}],"partialFingerprints":{"codehealthFindingId/v1":"4441551a91ba4746bc3fc0ac2afb5d5ad75fecd6e130014b5d80b7ea97483ce5"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/ProducerMessageImpl.cs"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"b7af47405b0a6090ea4c25cd7f8fcbbf0fca5315b2b3c3ceddb8445d3ca6cc74"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/QueueSender.cs"},"region":{"startLine":76}}}],"partialFingerprints":{"codehealthFindingId/v1":"67e4084eb470ca9bd53dee300347ddc83a041b98126e07487a4c50aac8070424"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/SingleEventLoop.cs"},"region":{"startLine":31}}}],"partialFingerprints":{"codehealthFindingId/v1":"72e45f7a0dc079587f3e15ca248e7ed9ec611e2a18850ddcd4c7ba3f6c00f7cc"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/TransportClient.cs"},"region":{"startLine":273}}}],"partialFingerprints":{"codehealthFindingId/v1":"f3fd57ef3b22426f57005c89d26d2ad84ff4fc0466f5a7783928b13f8942d34a"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Util/BatchExecutor.cs"},"region":{"startLine":76}}}],"partialFingerprints":{"codehealthFindingId/v1":"71ad780fd22e1ded1dc16c355eb464720aadc8d46972b2d0a44489b05c6b2b0b"}},{"ruleId":"D17","level":"error","message":{"text":"EmptyCatchBlock: empty catch block \u2014 the error is discarded with nothing recorded, so a failure here leaves no trace anywhere. Narrow the catch to the exception you actually expect, record it through whatever this codebase already uses to report problems, or \u2014 where swallowing really is correct, as it often is on a teardown/dispose path where throwing would mask the original failure \u2014 write down WHY in a comment on the catch. The comment has to give the reason: a note that only restates the swallow (\u0022ignored\u0022, \u0022do nothing\u0022) is read as no explanation at all and leaves this row in place. Any of the three makes the decision reviewable; all three clear this row."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Util/Scheduler.cs"},"region":{"startLine":51}}}],"partialFingerprints":{"codehealthFindingId/v1":"14cbaefe0c0db6000f52248599f60e4ca9b3fa0d10f5ea3114b20b536fda93aa"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(keli.wang): when one ack delay and blocks many other acks, can we just make this one a nack? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/NewQmq/AckHandlerQueue.cs"},"region":{"startLine":187}}}],"partialFingerprints":{"codehealthFindingId/v1":"ee95b96e59992ad82765e0d724ef773d82a9ff999d84461b1a5a34e8bb58b910"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(keli.wang): return list of messages or a special object? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/NewQmq/PullMessageDataTransformer.cs"},"region":{"startLine":35}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc7f9aff9050515e5911630784f2a697410d59f102338c4ed87624371d8ebe45"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(keli.wang): don\u0027t use request id here? \u2014 source code is not a task system: move the work to your tracker and leave a reference instead (e.g. \u0060// REF: #123\u0060), so the task is planned where tasks live and the ticket links back to the code."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/NewQmq/SendMessageDataTransformer.cs"},"region":{"startLine":21}}}],"partialFingerprints":{"codehealthFindingId/v1":"24f3e8e3e1866920a9c9fe391da1d8f8f7ca6f30203a2d971e95d1cf32a31541"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Util/ClientId.cs"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d91d8f82698f2f5f06df35f392ea8c842b378b66f6b5ecfecb981806b69d95d"}},{"ruleId":"D17","level":"warning","message":{"text":"CommentedOutCode: 3 consecutive commented-code lines"},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Util/PortSelector.cs"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"0de0cc8899d87e480a41a6a1632910bb71e8cc96a1a1590d652aff130d4371fe"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: BlockMessageException: NamedType BlockMessageException \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Exceptions/BlockMessageException.cs"},"region":{"startLine":3}}}],"partialFingerprints":{"codehealthFindingId/v1":"dadfdc219c41b16a7eb1de7c716fce2f471c3d8e72b4ee28d8094c2aac3d13a6"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: InitLenBuffer: Method InitLenBuffer \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Metainfo/MetainfoClient.cs"},"region":{"startLine":253}}}],"partialFingerprints":{"codehealthFindingId/v1":"55c10b9972527913356f452fba6be798a0a3e432d98432f040584523e1254858"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: ACKMessage: NamedType ACKMessage \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Model/ACKMessage.cs"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"b890ee9a5dd2eff88fcc85f5c1d3ca42cf1198ccba80531ed22ced3e7e57b207"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: HandlerMismatchException: NamedType HandlerMismatchException \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Model/HandlerMismatchException.cs"},"region":{"startLine":6}}}],"partialFingerprints":{"codehealthFindingId/v1":"16c63aa8bf57c90de60f428ce0f1e7a50c003c8c305198926696456965360e4c"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: QueryRequest: NamedType QueryRequest \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Model/QueryRequest.cs"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"b0775639a04609479957e6cf064eeb30119f4f9fe2e9080903f82b01ab07b0ad"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: ChannelContext: NamedType ChannelContext \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/ChannelContext.cs"},"region":{"startLine":5}}}],"partialFingerprints":{"codehealthFindingId/v1":"aadc20c32537c13398ed4ee0df29a326a8c46703598a0e0439c8fd0bf074d361"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: MarkWriterIndex: Method MarkWriterIndex \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/InputStream.cs"},"region":{"startLine":7}}}],"partialFingerprints":{"codehealthFindingId/v1":"1bb2edf21c3d7618704e4c7824c6187078a29e0de75c8fa6a433e8c33c17ac01"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: ResetWriterIndex: Method ResetWriterIndex \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/InputStream.cs"},"region":{"startLine":9}}}],"partialFingerprints":{"codehealthFindingId/v1":"4ab2c3d8b9b4a4bb83e17c848823be65104fdb9c017473af9a37922050ec7eb6"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: Limit: Method Limit \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Transport/InputStream.cs"},"region":{"startLine":15}}}],"partialFingerprints":{"codehealthFindingId/v1":"c552272dc7edfc5e2eb2fe6a9fad202024e7f0e433f476666f117ce4c114b17a"}},{"ruleId":"D17","level":"warning","message":{"text":"Dead code: ConcurrentPriorityQueue: NamedType ConcurrentPriorityQueue \u2014 no references found in solution."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Util/ConcurrentPriorityQueue.cs"},"region":{"startLine":11}}}],"partialFingerprints":{"codehealthFindingId/v1":"3ed46c7b531726846ba7345fc7646a2b204b863bd44b0ff08fe97cd7da8387be"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"20f30d9f0310efaedbdadc07125188b13c982cb303a19f3eff26d96a1bc4b2ec"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"c76214ea353eb4fc11b9a14413ac7adc8287ce1e4eeaebc80f15ae3a8fad91cb"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"2849d67d7cb833d377bbfc4ff945278b75ab35a09c7f16bb58390c3fae4f1dcc"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"800fee91c25fb04ece8eaf679f4f99a4ab08403c869707e3841979f2f10e5c1f"},"taxa":[{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"a32766a4cbff03642f62765bff11ffc76b7cbfc9d5828ffe62c41d38fd773dd9"},"taxa":[{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"c2a992aff8e473d0e4d5963117666be7f641a80aba0d19ec39ca5a813e466b63"},"taxa":[{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8710b3972834a37d0f7e6a229a4a9d0e7d9cf807c102082cf06584129d9e00a3"},"taxa":[{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"f3c474b694a7201a01f30cc90916dbdc11daed8100cf57518c5e104eb23ffa49"},"taxa":[{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"bb55dd891c6e8b368fa367f1a7199b2f5676ebc99b1fefe471a035ddf73b923a"},"taxa":[{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D30","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d301c6e60bb497a66228eb2bf5efd1ee217551f63f34886eeab5a740960f7aeb"}},{"ruleId":"D30","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0b290bc497fbe40aa63c0f19bccf57dd05a19c01598743b94aeaae0bca6b923b"}},{"ruleId":"D34","level":"note","message":{"text":"Dormant codebase: 36 of 36 significant files have no living knowledge \u2014 the codebase as a whole is dormant, not 36 separate risks. Re-engage owners or document before change."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"4c69f1e54b7dd6fe9029dd02df6ba8f8145b789f2f18dbdaa086c40ded49dba6"}},{"ruleId":"D34","level":"note","message":{"text":"Largest orphaned file: One of the largest files with no living knowledge remaining \u2014 a reasonable place to start a read-through before the aggregate risk above bites."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"clients/csharp/qmqclient/Util/Crc32.cs"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"e27594578c212a7936cac71dca20b85c3efff4bd8282de6756b225c40ae3fd14"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"9658270ba49f37ed04e99ab1263b6393cd42aed8bdfb7aafd9fa1070f5491895"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-1395","guid":"800e09e7-c11a-8654-9fa6-86f398995fed","name":"Dependency on Vulnerable Third-Party Component","shortDescription":{"text":"Dependency on Vulnerable Third-Party Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1395.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":15,"secretScannerRunsExcluded":0}},"redactionTokens":["A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."]}]}