{"$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":"D6","name":"Cohesion (LCOM4)","shortDescription":{"text":"Cohesion (LCOM4)"},"helpUri":"https://codehealth.canine.dev/dimensions/D6"},{"id":"D9","name":"Test Distribution","shortDescription":{"text":"Test Distribution"},"helpUri":"https://codehealth.canine.dev/dimensions/D9"},{"id":"D11","name":"Test Reliability","shortDescription":{"text":"Test Reliability"},"helpUri":"https://codehealth.canine.dev/dimensions/D11"},{"id":"D12","name":"Dependency Hygiene","shortDescription":{"text":"Dependency Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/D12"},{"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":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"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"]}},{"id":"D43","name":"Malicious Dependencies","shortDescription":{"text":"Malicious Dependencies"},"helpUri":"https://codehealth.canine.dev/dimensions/D43","relationships":[{"target":{"id":"CWE-506","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-506"]}},{"id":"D44","name":"Platform End-of-Life","shortDescription":{"text":"Platform End-of-Life"},"helpUri":"https://codehealth.canine.dev/dimensions/D44"},{"id":"AX10","name":"Code composition","shortDescription":{"text":"Code composition"},"helpUri":"https://codehealth.canine.dev/dimensions/AX10"},{"id":"AX3","name":"Project dependency cycles","shortDescription":{"text":"Project dependency cycles"},"helpUri":"https://codehealth.canine.dev/dimensions/AX3"},{"id":"AX4","name":"Dependency direction","shortDescription":{"text":"Dependency direction"},"helpUri":"https://codehealth.canine.dev/dimensions/AX4"},{"id":"AXB2","name":"Runtime readiness","shortDescription":{"text":"Runtime readiness"},"helpUri":"https://codehealth.canine.dev/dimensions/AXB2"},{"id":"M1","name":"Documentation (README)","shortDescription":{"text":"Documentation (README)"},"helpUri":"https://codehealth.canine.dev/dimensions/M1"},{"id":"M3","name":"Folder \u0026 project structure","shortDescription":{"text":"Folder \u0026 project structure"},"helpUri":"https://codehealth.canine.dev/dimensions/M3"},{"id":"M4","name":"Documentation accuracy","shortDescription":{"text":"Documentation accuracy"},"helpUri":"https://codehealth.canine.dev/dimensions/M4"},{"id":"P1","name":"CI/CD gates","shortDescription":{"text":"CI/CD gates"},"helpUri":"https://codehealth.canine.dev/dimensions/P1"},{"id":"P12","name":"CI test-gate honesty","shortDescription":{"text":"CI test-gate honesty"},"helpUri":"https://codehealth.canine.dev/dimensions/P12"},{"id":"P2","name":"Observability","shortDescription":{"text":"Observability"},"helpUri":"https://codehealth.canine.dev/dimensions/P2"},{"id":"P3","name":"Security \u0026 performance tooling","shortDescription":{"text":"Security \u0026 performance tooling"},"helpUri":"https://codehealth.canine.dev/dimensions/P3"},{"id":"P6","name":"Release Hygiene","shortDescription":{"text":"Release Hygiene"},"helpUri":"https://codehealth.canine.dev/dimensions/P6"},{"id":"X9","name":"Subsumed condition operand","shortDescription":{"text":"Subsumed condition operand"},"helpUri":"https://codehealth.canine.dev/dimensions/X9"}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"DeltaSharingService.listFiles (cyclomatic 27): DeltaSharingService.listFiles has cyclomatic complexity 27 (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":"server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala"},"region":{"startLine":452}}}],"partialFingerprints":{"codehealthFindingId/v1":"5ce038f6d1845d94ced4b6d7041655a00f0aa2dcb92bb799604a53c59637ca58"}},{"ruleId":"D1","level":"warning","message":{"text":"DeltaSharedTable.query (cyclomatic 25): DeltaSharedTable.query has cyclomatic complexity 25 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. 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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":338}}}],"partialFingerprints":{"codehealthFindingId/v1":"d87b8ee1a9381ebbb3061adcb7ee5b21fa9143ab8c4ab3f525b425376083ccbb"}},{"ruleId":"D1","level":"warning","message":{"text":"DeltaSharingCDCReader.queryCDF (cyclomatic 25): DeltaSharingCDCReader.queryCDF has cyclomatic complexity 25 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/standalone/internal/DeltaSharingCDCReader.scala"},"region":{"startLine":191}}}],"partialFingerprints":{"codehealthFindingId/v1":"372904e142b1f59579478baca90b0ba05a4f7db4d9c81ef4ccba4f53aed45dbe"}},{"ruleId":"D1","level":"warning","message":{"text":"PartitionFilterUtils.isSupportedExpression (cyclomatic 19): PartitionFilterUtils.isSupportedExpression has cyclomatic complexity 19 (threshold 15). To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Where every arm is uniform \u2014 the same kind of value, with no behaviour of its own \u2014 a table keyed by the case is the shorter form; wherever the arms carry different data or different behaviour, keep them as cases, because collapsing those trades an explicit, reviewable set of cases for nothing."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/standalone/internal/PartitionFilterUtils.scala"},"region":{"startLine":75}}}],"partialFingerprints":{"codehealthFindingId/v1":"6880ff5cee062ddaec4522297417ffd3ddd697c12ccce3e522f255316b003162"}},{"ruleId":"D1","level":"warning","message":{"text":"DeltaSharedTable.queryCDF (cyclomatic 17): DeltaSharedTable.queryCDF has cyclomatic complexity 17 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. 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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":697}}}],"partialFingerprints":{"codehealthFindingId/v1":"7f863e0e3853d18ded445032690718228dcc576ae71eaef858252a1c7320847b"}},{"ruleId":"D1","level":"warning","message":{"text":"DeltaSharedTable.queryDataChangeSinceStartVersion (cyclomatic 16): DeltaSharedTable.queryDataChangeSinceStartVersion 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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":557}}}],"partialFingerprints":{"codehealthFindingId/v1":"bee46f357be670b81348a4b9056846ae49d462d479a8f2624d7ca68b9d8e910d"}},{"ruleId":"D1","level":"warning","message":{"text":"DeltaSharingReader._to_pandas (cyclomatic 16): DeltaSharingReader._to_pandas 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":"python/delta_sharing/reader.py"},"region":{"startLine":525}}}],"partialFingerprints":{"codehealthFindingId/v1":"e2c528891775fe9c98977365db05bf8697924c66f4dfc18a308638f85535e193"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharedTable.query (cognitive 46): DeltaSharedTable.query has cognitive complexity 46 (threshold 15). Drivers by points: if/else 28 (36 pts), boolean chains 6, error handling 2 (4 pts) (nesting depth added 10). 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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":338}}}],"partialFingerprints":{"codehealthFindingId/v1":"8ac507db813989c0442269144080787caf1c651ef4cc5f8517dc715db7058c28"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingService.listFiles (cognitive 37): DeltaSharingService.listFiles has cognitive complexity 37 (threshold 15). Drivers by points: if/else 18 (27 pts), boolean chains 10 (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":"server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala"},"region":{"startLine":452}}}],"partialFingerprints":{"codehealthFindingId/v1":"e31c109af6ff404e2c998a04b35cc629c2ed6064f8ae3c934e21dfb8a721c7bc"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingReader._to_pandas (cognitive 32): DeltaSharingReader._to_pandas has cognitive complexity 32 (threshold 15). Drivers by points: if/else 13 (25 pts), loops 3 (4 pts), ternaries 1 (2 pts), boolean chains 1 (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":"python/delta_sharing/reader.py"},"region":{"startLine":525}}}],"partialFingerprints":{"codehealthFindingId/v1":"eefd7fc595bf9fb323cf5623274a2a6378c98fcd0862e4473a6e5a70c35efb62"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingCDCReader.queryCDF (cognitive 30): DeltaSharingCDCReader.queryCDF has cognitive complexity 30 (threshold 15). Drivers by points: if/else 13 (14 pts), match/switch 5 (10 pts), boolean chains 6 (nesting depth added 6). 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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharingCDCReader.scala"},"region":{"startLine":191}}}],"partialFingerprints":{"codehealthFindingId/v1":"5d1b1bd6b52a4481b984091169cf5e5c5e161e16f9be46f83dd90bd28472e299"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingSource.getTableFileChanges (cognitive 27): DeltaSharingSource.getTableFileChanges has cognitive complexity 27 (threshold 15). Drivers by points: if/else 10 (16 pts), match/switch 2 (5 pts), boolean chains 4, loops 1 (2 pts) (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":"client/src/main/scala/io/delta/sharing/spark/DeltaSharingSource.scala"},"region":{"startLine":450}}}],"partialFingerprints":{"codehealthFindingId/v1":"06c515a47442a1d8995e365d78c32a38607af72b9f2e26c0557dd4edc955cf9f"}},{"ruleId":"D2","level":"warning","message":{"text":"RandomAccessHttpInputStream.reopen (cognitive 26): RandomAccessHttpInputStream.reopen has cognitive complexity 26 (threshold 15). Drivers by points: if/else 13 (19 pts), error handling 1 (3 pts), match/switch 1 (3 pts), boolean chains 1 (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":"client/src/main/scala/io/delta/sharing/client/RandomAccessHttpInputStream.scala"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"849121e36b2158a436ff5825c6f04b14ecbfaf638a9adf3d6bdc35d89dd42238"}},{"ruleId":"D2","level":"warning","message":{"text":"CachedTableManager.handleQuerySpecificCachedTableRefresh (cognitive 23): CachedTableManager.handleQuerySpecificCachedTableRefresh has cognitive complexity 23 (threshold 15). Drivers by points: if/else 11 (19 pts), error handling 1 (3 pts), match/switch 1 (nesting depth added 10). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/org/apache/spark/delta/sharing/PreSignedUrlCache.scala"},"region":{"startLine":280}}}],"partialFingerprints":{"codehealthFindingId/v1":"f0287ab0d6b5d5c7e2930765345ce2835e914c5647e22604cc0c161b8dc35a9b"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharedTableKernel.query (cognitive 23): DeltaSharedTableKernel.query has cognitive complexity 23 (threshold 15). Drivers by points: if/else 11 (15 pts), boolean chains 4, loops 1 (2 pts), match/switch 1 (2 pts) (nesting depth added 6). 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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":260}}}],"partialFingerprints":{"codehealthFindingId/v1":"597c99a98e07232a11d562222668341df33616939dbf63e58a2680d5f7724287"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingRestClient.getResponseWithRetries (cognitive 22): DeltaSharingRestClient.getResponseWithRetries has cognitive complexity 22 (threshold 15). Drivers by points: if/else 8 (12 pts), boolean chains 5, loops 1 (3 pts), error handling 1 (2 pts) (nesting depth added 7). 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":"client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala"},"region":{"startLine":1399}}}],"partialFingerprints":{"codehealthFindingId/v1":"fb9803b3c1561c8b482930e24f22cf41f3f8febcfb2c505573b30deb40bfb585"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingSource.getCDFFileChanges (cognitive 21): DeltaSharingSource.getCDFFileChanges has cognitive complexity 21 (threshold 15). Drivers by points: boolean chains 7, if/else 6 (7 pts), match/switch 2 (6 pts), loops 1 (nesting depth added 5). 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":"client/src/main/scala/io/delta/sharing/spark/DeltaSharingSource.scala"},"region":{"startLine":603}}}],"partialFingerprints":{"codehealthFindingId/v1":"b28fde578c4dd9536a8912f7b150e297f7bf1b2bead0e930fb39162016b2e30e"}},{"ruleId":"D2","level":"warning","message":{"text":"CachedTableManager.register (cognitive 20): CachedTableManager.register has cognitive complexity 20 (threshold 15). Drivers by points: if/else 11 (16 pts), boolean chains 1, error handling 1, loops 1, match/switch 1 (nesting depth added 5). 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":"client/src/main/scala/org/apache/spark/delta/sharing/PreSignedUrlCache.scala"},"region":{"startLine":622}}}],"partialFingerprints":{"codehealthFindingId/v1":"d64bbab45d6b31eee254e7d8de6d29fda9369a03d9c514365d178374c68b4a98"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingReader._normalize_record_batch (cognitive 20): DeltaSharingReader._normalize_record_batch has cognitive complexity 20 (threshold 15). Drivers by points: if/else 7 (17 pts), boolean chains 2, loops 1 (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":"python/delta_sharing/reader.py"},"region":{"startLine":630}}}],"partialFingerprints":{"codehealthFindingId/v1":"b694933431ef2051183d1a7902675fcd0abdaab32c87acc83a6ea12291af32b1"}},{"ruleId":"D2","level":"warning","message":{"text":"CachedTableManager.registerQuerySpecificCachedTable (cognitive 19): CachedTableManager.registerQuerySpecificCachedTable has cognitive complexity 19 (threshold 15). Drivers by points: if/else 11 (14 pts), match/switch 2 (4 pts), boolean chains 1 (nesting depth added 5). 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":"client/src/main/scala/org/apache/spark/delta/sharing/PreSignedUrlCache.scala"},"region":{"startLine":472}}}],"partialFingerprints":{"codehealthFindingId/v1":"9603e91c7601ff3dfee5acc50f71146672d05784e5c72de29403d843affea761"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharedTableKernel.getSharedTableSnapshot (cognitive 19): DeltaSharedTableKernel.getSharedTableSnapshot has cognitive complexity 19 (threshold 15). Drivers by points: if/else 8 (11 pts), error handling 4 (8 pts) (nesting depth added 7). 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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":137}}}],"partialFingerprints":{"codehealthFindingId/v1":"42dd14516bedda4baf4687e865962d2d6d3ac1803bd7c189cf769f97812cd2b1"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharedTable.queryCDF (cognitive 19): DeltaSharedTable.queryCDF has cognitive complexity 19 (threshold 15). Drivers by points: if/else 12 (15 pts), boolean chains 3, match/switch 1 (nesting depth added 3). 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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":697}}}],"partialFingerprints":{"codehealthFindingId/v1":"23cc5ca27fdcac3a6320916d2ab66120478c59bf6e71d1acabb735f9d50b0275"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharedTable.queryDataChangeSinceStartVersion (cognitive 17): DeltaSharedTable.queryDataChangeSinceStartVersion has cognitive complexity 17 (threshold 15). Drivers by points: if/else 9 (13 pts), boolean chains 3, match/switch 1 (nesting depth added 4). 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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":557}}}],"partialFingerprints":{"codehealthFindingId/v1":"61b72d82e15984575742a53af80817fdc2caa45174de2d5c978b40ce42387e54"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingRestClient.parsePath (cognitive 16): DeltaSharingRestClient.parsePath has cognitive complexity 16 (threshold 15). Drivers by points: if/else 9 (11 pts), boolean chains 5 (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, 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":"client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala"},"region":{"startLine":1610}}}],"partialFingerprints":{"codehealthFindingId/v1":"1b02c4ec27bfefcea9e3525551c694e729fbbeeb01231fee94d37d0f769a5170"}},{"ruleId":"D2","level":"warning","message":{"text":"UnsupportedOpPruner.pruneRecurse (cognitive 16): UnsupportedOpPruner.pruneRecurse has cognitive complexity 16 (threshold 15). Drivers by points: if/else 8 (13 pts), match/switch 2 (3 pts) (nesting depth added 6). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/filters/UnsupportedOpPruner.scala"},"region":{"startLine":40}}}],"partialFingerprints":{"codehealthFindingId/v1":"07ba1d1666b69c8e366550259a59dd08f9a9d178e7ef522450906df06440c5f9"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingCDFReader.getMinUrlExpiration (cognitive 16): DeltaSharingCDFReader.getMinUrlExpiration has cognitive complexity 16 (threshold 15). Drivers by points: if/else 10 (13 pts), boolean chains 3 (nesting depth added 3). 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":"client/src/main/scala/io/delta/sharing/spark/RemoteDeltaCDFRelation.scala"},"region":{"startLine":151}}}],"partialFingerprints":{"codehealthFindingId/v1":"d8c50aa6d09cb358a45e44345bc13fce08225ce3f2b15b6383b6262b4730aee7"}},{"ruleId":"D2","level":"warning","message":{"text":"DeltaSharingProfile.from_json (cognitive 16): DeltaSharingProfile.from_json has cognitive complexity 16 (threshold 15). Drivers by points: if/else 8 (13 pts), boolean chains 3 (nesting depth added 5). 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":"python/delta_sharing/protocol.py"},"region":{"startLine":72}}}],"partialFingerprints":{"codehealthFindingId/v1":"97f8b51fd9d0a92a9061ecf04e4932b7543ba00b57f41ded62fe7444aa81fd3e"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: client/DeltaSharingClient.scala: FileTooLong \u2014 1273 significant lines (blank, comment-only and punctuation-only lines excluded), about 75% of them inside a single declaration: DeltaSharingRestClient (199-1552). The bar is 500 significant lines; this is 773 over it, 2.55\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"67fc767db2fd4a7a6b08dbf60dc988a12937cd8ab7dff6ce5bdcb44eca94bd76"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: ConfUtils: TooManyMethods \u2014 50 methods. The bar is 30 methods; this is 20 over it, 1.67\u00D7 the bar. 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":"client/src/main/scala/io/delta/sharing/client/util/ConfUtils.scala"},"region":{"startLine":25}}}],"partialFingerprints":{"codehealthFindingId/v1":"27bf560420ac58020d69da9ba3c2c208c9b3c59c76a06cfcfd6bcd12e29b2089"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: server/DeltaSharingService.scala: FileTooLong \u2014 704 significant lines (blank, comment-only and punctuation-only lines excluded), about 63% of them inside a single declaration: DeltaSharingService (190-775). The bar is 500 significant lines; this is 204 over it, 1.41\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"a0067e91f7e0c615e0bab95ec0c0b00eea2a8d72a4c60fe5d22ac2b55d260bf3"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: internal/DeltaSharedTable.scala: FileTooLong \u2014 697 significant lines (blank, comment-only and punctuation-only lines excluded), about 94% of them inside a single declaration: DeltaSharedTable (69-982). The bar is 500 significant lines; this is 197 over it, 1.39\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"3c2e90794c81b27c35fc8462a610682637dc5c494971bdf439dd211a8b32acd8"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: spark/DeltaSharingSource.scala: FileTooLong \u2014 672 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 172 over it, 1.34\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/spark/DeltaSharingSource.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"5375de458a63cb907448cad94b8ce576d00900c5073e152c8400c1755bb8b04c"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: internal/DeltaSharedTableKernel.scala: FileTooLong \u2014 575 significant lines (blank, comment-only and punctuation-only lines excluded), about 88% of them inside a single declaration: DeltaSharedTableKernel (65-802). The bar is 500 significant lines; this is 75 over it, 1.15\u00D7 the bar. Moving the declarations that sit BESIDE it into sibling files will not shorten this file. Extract from INSIDE that declaration instead: lift each cohesive group of its body \u2014 the parts that share the same inputs and are named together \u2014 into its own unit in a sibling file, and have the original call them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"92b8fe2228a0e88fb6a59d88b68a6a5cc50857d5c6f771f318722a716b08505e"}},{"ruleId":"D3","level":"warning","message":{"text":"TooManyMethods: DeltaSharingRestClient: TooManyMethods \u2014 34 methods. The bar is 30 methods; this is 4 over it, 1.13\u00D7 the bar. 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":"client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala"},"region":{"startLine":199}}}],"partialFingerprints":{"codehealthFindingId/v1":"2f270931e213c8832c6660afa4ac1d7b2a22b94cd4c80b935d662b19e9d752ea"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: delta_sharing/reader.py: FileTooLong \u2014 515 significant lines (blank, comment-only and punctuation-only lines excluded). The bar is 500 significant lines; this is 15 over it, 1.03\u00D7 the bar. To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"python/delta_sharing/reader.py"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e7f5c9669df24a086e27b474a482782aa69ffe57b6b14f5d9d49ceb272f672d"}},{"ruleId":"D4","level":"warning","message":{"text":"Members sharing a duplicated core (4 members, 50\u002B identical tokens): client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:324-357 | client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:360-390 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:368-401 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:404-434 \u2014 These 4 members share a duplicated core: a run of at least 50 identical tokens appears in every one of them. That run is NOT broken out as duplicated-block rows below \u2014 it is what admitted this row, and the blocks below cover only the part of it that clears the block floor, so they understate the correspondence. Read the members as one construct written 4 times. The repair is at the members\u0027 grain \u2014 factor the shared implementation out once and have all of them call it with their differences as parameters or as an injected step, or, where the difference is systematic, generate them from one template. Extracting the individual blocks below is not the same fix: it leaves every body in place and the next edit still has to be made 4 times."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":324}}}],"partialFingerprints":{"codehealthFindingId/v1":"f842495fd4cc56e891bf618e895303a273f2c9953d2db4143b4f239d016392f7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (28 lines \u00D7 2): client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:330-357 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:374-401 \u2014 before extracting anything, compare \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala\u0060 and \u0060server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 93 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:330\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, 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":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":330}}}],"partialFingerprints":{"codehealthFindingId/v1":"4a3f36aab818af16a334144c22fb942b179ea278f57f716032819395c588830a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (25 lines \u00D7 2): client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:366-390 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:410-434 \u2014 before extracting anything, compare \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala\u0060 and \u0060server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 93 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:366\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, 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":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":366}}}],"partialFingerprints":{"codehealthFindingId/v1":"efd60df6fac1b6714d550d1156a8f669009fc40715c93b864086148556212d99"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (22\u201324 lines \u00D7 2): client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:612-633 | client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:793-816 \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 \u0060client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:612\u0060 it runs out through the closing brace of the declaration holding it and carries on into the declaration that follows \u2014 the window is the tail of one member plus the head of the next, so no call can be substituted for those exact lines, and the smallest declaration that contains all of them is the type they sit in. The repeated unit is the member each site sits in: where those members\u0027 bodies are the same, move one whole member to the shared location and have the others delegate to it; where the copies are a run of near-identical overloads or wrappers that differ only in their signatures, the repetition IS the run \u2014 a one-line delegation has no helper inside it to lift \u2014 so generate the run from the set it enumerates, or accept it and keep each member\u0027s own documentation with it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala"},"region":{"startLine":612}}}],"partialFingerprints":{"codehealthFindingId/v1":"9703698cc3f706fd7e3b3fa87fb698fbe3a2b4323b019ca6c31132da6d301ead"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (15\u201320 lines \u00D7 2): client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:640-659 | client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:826-840 \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 \u0060client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:640\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":"client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala"},"region":{"startLine":640}}}],"partialFingerprints":{"codehealthFindingId/v1":"777f00623db20a55d2f9889aaa4dd149f78827c21b4934c8457ad743de49bb98"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:547-564 | server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala:964-981 \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 \u0060server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:547\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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":547}}}],"partialFingerprints":{"codehealthFindingId/v1":"5cff266e9e684e209f51cd00fd4562652787ca21ab7aadfd9448d1d01e0ef417"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 4): client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:325-340 | client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:361-376 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:369-384 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:405-420 \u2014 before extracting anything, compare \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala\u0060 and \u0060server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 93 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:325\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":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":325}}}],"partialFingerprints":{"codehealthFindingId/v1":"70c963eb9909be0f1fb81ff872dd845e861b4e1efad5e6f0ebab1267d7a5f6fa"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 3): server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:547-562 | server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala:936-951 | server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala:964-979 \u2014 there are 3 copies across 2 file(s) \u2014 more copies than files, so at least one file holds the block twice. Extract it once into a single shared function every call site can reach and call it from all 3 sites; resolving a subset leaves the remainder to drift apart. Read the line range as the matched WINDOW rather than a finished unit: at \u0060server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:547\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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":547}}}],"partialFingerprints":{"codehealthFindingId/v1":"b9a2cc409ef656711aeea90ef5923d76b046d879d2449ec9e929b6b154642c47"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 2): client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:742-757 | client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:894-909 \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 \u0060client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala:742\u0060 it runs out through the closing brace of the declaration holding it \u2014 the window is that declaration\u0027s tail, not a fragment that begins part-way through something, 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":"client/src/main/scala/io/delta/sharing/client/DeltaSharingClient.scala"},"region":{"startLine":742}}}],"partialFingerprints":{"codehealthFindingId/v1":"2be550d6cf00023c0e7217fe652dc153f1de8b792e4fc4265a96fbfb12e32f25"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (14 lines \u00D7 2): client/src/main/scala/io/delta/sharing/client/model.scala:103-116 | server/src/main/scala/io/delta/sharing/server/model.scala:67-80 \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 \u0060client/src/main/scala/io/delta/sharing/client/model.scala:103\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":"client/src/main/scala/io/delta/sharing/client/model.scala"},"region":{"startLine":103}}}],"partialFingerprints":{"codehealthFindingId/v1":"c6730fb7a6f016519b55d1cbcc64bc509ae0ab998147d23e1a8b16456876e2cc"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala:555-567 | server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala:574-586 \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":"server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala"},"region":{"startLine":555}}}],"partialFingerprints":{"codehealthFindingId/v1":"fd66c37564795ce91cfa5c49bba59c6252cbc337cc75ba1f803940eadb7ce93c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 3): server/src/main/scala/io/delta/standalone/internal/JsonPredicateEvaluatorV2.scala:215-224 | server/src/main/scala/io/delta/standalone/internal/JsonPredicateEvaluatorV2.scala:253-263 | server/src/main/scala/io/delta/standalone/internal/JsonPredicateEvaluatorV2.scala:304-314 \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 \u0060server/src/main/scala/io/delta/standalone/internal/JsonPredicateEvaluatorV2.scala:215\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":"server/src/main/scala/io/delta/standalone/internal/JsonPredicateEvaluatorV2.scala"},"region":{"startLine":215}}}],"partialFingerprints":{"codehealthFindingId/v1":"4cb6925724eaa3416d29a4034f5ca1d055249395ced035b67166c37bb416a6ea"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:691-699 | server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala:908-916 \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. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":691}}}],"partialFingerprints":{"codehealthFindingId/v1":"2ce47e052d4a4d719427574e379bdd6c72f138e8315037c3a4669b6042a35c10"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:121-128 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:148-155 \u2014 before extracting anything, compare \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala\u0060 and \u0060server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 93 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":121}}}],"partialFingerprints":{"codehealthFindingId/v1":"456de50d3c26ce01dc8b21ec8603655b7ed14b727ab667422675f64b24fa63a3"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:147-154 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:184-191 \u2014 before extracting anything, compare \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala\u0060 and \u0060server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 93 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":147}}}],"partialFingerprints":{"codehealthFindingId/v1":"af93826ebc656439e46052ec99c28968b86c795b20327fee1e316456db878c36"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala:167-174 | server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala:204-211 \u2014 before extracting anything, compare \u0060client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala\u0060 and \u0060server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala\u0060 as WHOLE FILES: this scan already matched 6 separate duplicated blocks between them, totalling at least 93 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":167}}}],"partialFingerprints":{"codehealthFindingId/v1":"6102f9b098660bc773fd402c769e45ba28399fe213c1bbf721ce2436a83d7f29"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:113-123 | server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala:120-130 \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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":113}}}],"partialFingerprints":{"codehealthFindingId/v1":"a2fbe5e211f0bc5be53efce7e213bc37e431f061d1c4edec8fa41e51416686e6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:473-480 | server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala:918-925 \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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":473}}}],"partialFingerprints":{"codehealthFindingId/v1":"df10d6010026fd41a71cfa721b2c9c22be9e6af7480fe8f0a28fbee7f7a44d0c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala:681-688 | server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala:898-905 \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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":681}}}],"partialFingerprints":{"codehealthFindingId/v1":"f9b3ea91c05608842010c0aab0873c77bf2fc4d0d147632bf1f7825c24409abf"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): server/src/main/scala/io/delta/sharing/server/model.scala:222-233 | server/src/main/scala/io/delta/standalone/internal/model.scala:44-55 \u2014 before extracting anything, compare \u0060server/src/main/scala/io/delta/sharing/server/model.scala\u0060 and \u0060server/src/main/scala/io/delta/standalone/internal/model.scala\u0060 as WHOLE FILES: 89% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 2 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/sharing/server/model.scala"},"region":{"startLine":222}}}],"partialFingerprints":{"codehealthFindingId/v1":"4a41e852b4214aef8b868835dfe69db65fd7e2a6ef017754186003341171c701"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): server/src/main/scala/io/delta/sharing/server/model.scala:292-302 | server/src/main/scala/io/delta/standalone/internal/model.scala:114-124 \u2014 before extracting anything, compare \u0060server/src/main/scala/io/delta/sharing/server/model.scala\u0060 and \u0060server/src/main/scala/io/delta/standalone/internal/model.scala\u0060 as WHOLE FILES: 89% of the shorter file\u0027s lines also appear in the other, so this reads as one file having been copied from the other rather than as a helper waiting to be extracted. The 2 duplicated block(s) this scan matched between them are fragments of that copy, not the extent of it \u2014 treat the file pair as the unit. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Note first that the copies are not typed on the same thing: the declarations holding them bind \u0060protocol\u0060 to \u0060DeltaFormatResponseProtocol = null\u0060 in one and \u0060DeltaResponseProtocol = null\u0060 in another, and the duplicated lines use it. The extracted unit therefore needs a parameter type that fits BOTH \u2014 their common supertype where they have one, or a new abstraction over them where they do not \u2014 and settling that is the step that comes BEFORE the extraction above. Where the two types are deliberately unrelated, the duplication is the price of that separation and the honest resolution is to record the decision rather than to extract."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/sharing/server/model.scala"},"region":{"startLine":292}}}],"partialFingerprints":{"codehealthFindingId/v1":"20acbe0147fb594f7dc320ab0a35c5c1ce68f2202b029e56e8bb709a1ab5c021"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (26\u201328 lines \u00D7 2): python/delta_sharing/delta_sharing.py:127-154 | python/delta_sharing/delta_sharing.py:167-192 \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 \u0060python/delta_sharing/delta_sharing.py:127\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. 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":"python/delta_sharing/delta_sharing.py"},"region":{"startLine":127}}}],"partialFingerprints":{"codehealthFindingId/v1":"e77992dd098e2ee7cf9247ff6530609d9e759a064757ea686933b28ca3405046"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): python/delta_sharing/protocol.py:259-270 | python/delta_sharing/protocol.py:274-285 \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 \u0060python/delta_sharing/protocol.py:259\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. 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":"python/delta_sharing/protocol.py"},"region":{"startLine":259}}}],"partialFingerprints":{"codehealthFindingId/v1":"3da2e98c9e9fb8314ea96b8efc4b7fddd578ee4f9497606c599f7c7cee36eac1"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): python/delta_sharing/rest_client.py:228-235 | python/delta_sharing/rest_client.py:266-273 \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":"python/delta_sharing/rest_client.py"},"region":{"startLine":228}}}],"partialFingerprints":{"codehealthFindingId/v1":"b21a53e857b053e908b2af71901e4cc82519e5c87fcbc1c175fc7afb8c7e14f7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (22 lines \u00D7 2): client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala:43-64 | client/src/main/scala-2.13/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala:70-91 \u2014 before extracting anything, compare \u0060client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala\u0060 and \u0060client/src/main/scala-2.13/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 45 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala:43\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":"client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"2453dc5d2e43780077bfc6a9e8119ee90c1f518fdeddcadbb719fb5f841fd636"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (18 lines \u00D7 2): client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala:92-109 | client/src/main/scala-2.13/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala:119-136 \u2014 before extracting anything, compare \u0060client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala\u0060 and \u0060client/src/main/scala-2.13/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 45 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place. Each matched range is the entire body of the declaration above it, so the region is already a complete unit: move that whole declaration to the shared location and have each site call it, rather than lifting the lines out of their bodies. Any \u0060return\u0060 inside it is the body\u0027s own exit and keeps its meaning in the moved unit."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala"},"region":{"startLine":92}}}],"partialFingerprints":{"codehealthFindingId/v1":"ab0f5df10d123603b3fade9da805143663b3d0b012c93a9468d6f90739c5c00a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala:34-38 | client/src/main/scala-2.13/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala:34-38 \u2014 before extracting anything, compare \u0060client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala\u0060 and \u0060client/src/main/scala-2.13/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala\u0060 as WHOLE FILES: this scan already matched 3 separate duplicated blocks between them, totalling at least 45 lines, which is the signature of one file having been copied from the other rather than of a helper waiting to be extracted. The two sit in different directories, so one cannot simply be deleted in favour of the other while both are reached separately: hoist the shared part into a location both already depend on and have each file call it, and retire whichever file turns out to have no caller of its own left. Extracting one helper per block leaves the fork in place."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/src/main/scala-2.12/io/delta/sharing/spark/perf/DeltaSharingLimitPushDown.scala"},"region":{"startLine":34}}}],"partialFingerprints":{"codehealthFindingId/v1":"97b0dfbfdb8d0c24198f751a6ce688e4a2d94a2c39b3c4fb6f2625b92b75cc79"}},{"ruleId":"D13","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":"9e35a8bd81ccd8e5484182324e9adaf5b64587c88db242a7d2a40a9ba2f6b42f"},"taxa":[{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: python/delta_sharing/reader.py: python/delta_sharing/reader.py changed 2 times in last 90 days, and the most complex body those changes touched has cyclomatic complexity 16 in DeltaSharingReader._to_pandas at line 525. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, with the area under test before it moves. Counted over 2026-05-30..2026-08-28, the 90 days ending at the analysed commit. Reproduce with \u0060git log --since=\u00272026-05-30 12:53:43 -07:00\u0027 --until=\u00272026-08-28 12:53:43 -07:00\u0027 --full-history --no-merges -- python/delta_sharing/reader.py\u0060: merges are excluded because a merge re-states changes already counted at their own commits, and history is NOT path-simplified because a change that reached the file through a merged branch is still a change to it. That command counts raw commits and can read HIGHER than this row, which counts a cherry-picked re-land, and a revert together with the commit it undoes, once each \u2014 a difference of several commits on a file whose history was re-landed or reverted inside the window."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"python/delta_sharing/reader.py"},"region":{"startLine":525}}}],"partialFingerprints":{"codehealthFindingId/v1":"0b06acb80ae4c9667976850867bebb98300422fa26cb781555f852d780d32a69"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 3 significant file(s) lose their only recent owner: client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala, client/src/main/scala/io/delta/sharing/filters/OpConverter.scala, client/src/main/scala/io/delta/sharing/filters/UnsupportedOpPruner.scala. Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"21b74d1e027b301f0c8589b2fdafdd9411f0d61dc5354feb7d73a84ba35669a5"}},{"ruleId":"D16","level":"note","message":{"text":"Further sole-owners (lower concentration): 3 other contributor(s) are each the sole owner of a small amount of code below the off-boarding threshold \u2014 folded into the bus-factor score and metrics (7 single-owned of 65 analysed files in total, counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 65 of the 84 production source files in this repository met that bar). They are anonymized user #2 (2 file(s)), anonymized user #3 (1 file(s)), anonymized user #4 (1 file(s)) \u2014 spread or document their files in the same way, at lower priority than the named off-boarding risks above."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"6eac528f2429e724e1a97ed868f79eefbcf1888dab4af9a9e673429369bb5b2f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(abhijit): For literal operations, we can optimize evaluation by caching \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":"client/src/main/scala/io/delta/sharing/filters/JsonPredicates.scala"},"region":{"startLine":345}}}],"partialFingerprints":{"codehealthFindingId/v1":"d72ad13766ca324bee983b3a548af42e37ba8973866bb8ab583eecff7058cb26"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO(abhijit): For literal operations, we can optimize evaluation by caching \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":"server/src/main/scala/io/delta/sharing/server/common/JsonPredicates.scala"},"region":{"startLine":389}}}],"partialFingerprints":{"codehealthFindingId/v1":"b5ead21c426c7dcffc69a8986f4780c3f73aab0e8e4417c9b83f36621a6fea9f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: # TODO: Support use_delta_format once load_as_spark can pass it through. \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":"python/delta_sharing/delta_sharing.py"},"region":{"startLine":250}}}],"partialFingerprints":{"codehealthFindingId/v1":"6b1418475d1b44af6094ea8fcff4b9ca1111c48136ff354ce63c9e15c1f6aa9e"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: # TODO: Support use_delta_format once load_table_changes_as_spark can pass it through. \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":"python/delta_sharing/delta_sharing.py"},"region":{"startLine":316}}}],"partialFingerprints":{"codehealthFindingId/v1":"0aaeef8ebee8526bf9926e1a2d6fea74415a2cb1787fd2f09a320e1b6ffc7837"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: # TODO: Enable once timestampntz \u002B CDF support is enabled for both \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":"python/delta_sharing/tests/test_delta_sharing.py"},"region":{"startLine":1752}}}],"partialFingerprints":{"codehealthFindingId/v1":"b8cf9fc272cb29f137116b8380ab73c2eeea67531609d167f13a3ddf6e1bfee0"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: # TODO: enable once delta-kernel-rs supports schema changes during version range \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":"python/delta_sharing/tests/test_delta_sharing.py"},"region":{"startLine":2574}}}],"partialFingerprints":{"codehealthFindingId/v1":"451b4220cf34024d0a0ffd1e03bc59e240d95958f9a26310145c66ceabbfe451"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: # TODO: enable once delta-kernel-rs supports schema changes during version range \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":"python/delta_sharing/tests/test_delta_sharing.py"},"region":{"startLine":2743}}}],"partialFingerprints":{"codehealthFindingId/v1":"e61b11944499069682bea7836b361e07d3a38c546133a09ae889c3a345eefd9f"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: stop supporting this format once all clients are migrated. \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":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":793}}}],"partialFingerprints":{"codehealthFindingId/v1":"5f125170b9fe74c85a13babcdea39dfa85938984c825a489e2ee77b57115a3cb"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: deprecate HEAD request in favor of the GET request \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":"server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala"},"region":{"startLine":276}}}],"partialFingerprints":{"codehealthFindingId/v1":"fcb0493b12179f7e3be1b5f88d66e5e3c5aa6829ea1d1fbd58cd5bef2e1797c6"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO Support \u0060limitHint\u0060 \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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":357}}}],"partialFingerprints":{"codehealthFindingId/v1":"0e9f51f14054c627ef129d8d67030873b974a9991d281e2449b930c18b86c36b"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO Open the \u0060state\u0060 field in Delta Standalone library. \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":"server/src/main/scala/io/delta/standalone/internal/DeltaSharedTable.scala"},"region":{"startLine":414}}}],"partialFingerprints":{"codehealthFindingId/v1":"ea7d893e1bd85644a051a1b9e3031a87559f2f6d214dc229f9b25927369c2f44"}},{"ruleId":"D17","level":"warning","message":{"text":"TodoComment: // TODO: test with different Trigger.xx: \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":"spark/src/test/scala/io/delta/sharing/spark/DeltaSharingSourceSuite.scala"},"region":{"startLine":41}}}],"partialFingerprints":{"codehealthFindingId/v1":"f008358cd442166689e2b4f9d7374227dd545f398ef3dbf8499801ebbcfc0054"}},{"ruleId":"D26","level":"note","message":{"text":"Projects may be oversized for their cohesion: 1 of 1 project(s) overshoot their size bounds, lowering Project Cohesion to 0.0/10. The most over is \u0060(repository root)\u0060 (15357 LoC, 236 public types across 21 directories). Review these for cohesion \u2014 draw the boundary inside the module first (group each responsibility into its own package or directory and keep the cross-boundary members non-public), since splitting a published package moves types between packages and breaks consumers."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d5a94650dc74886a0f1f08eb9fb6775f1fc395279ef7e901c970c9d26f767a72"}},{"ruleId":"D28","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":"98d4bc6d572f6eb5493df2bd6e87dd765cb01ef89194757e9c60412bdcbfca4f"},"properties":{"commitSha":"8499a811d5f11241696bb8d6db69697046253d02"}},{"ruleId":"D28","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":"e6f19a94c7ccaaaa62e602e7221f6a8f491a24e31a5e70d308a3e6507fa09a9c"},"properties":{"commitSha":"a14439945b6876ca961c54f366f38b10f6b13ae0"}},{"ruleId":"D28","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":"31235e937800487665e287cc5247ca4019a377334d0ce57725918ede466001b9"},"properties":{"commitSha":"69ad66aa304ba8641ebe947aca950e9dd267d762"}},{"ruleId":"D28","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":"5257a051ed7fc7e176a880e41f0ac8aec9d7e9af50807c3243e6fd54f97ab912"},"properties":{"commitSha":"e16381db5ad7b25dd1516f9adab78872111a5297"}},{"ruleId":"D28","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":"18ad496609abd48d542f3d849f6d1c41efac07137b2b3a757fcdb4ec8d2939bf"},"properties":{"commitSha":"e16381db5ad7b25dd1516f9adab78872111a5297"}},{"ruleId":"D28","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":"3cdfd7beb31b791ae18dcb425e21eb157c842e7bbcd522cc330573c1a6538db1"}},{"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":"d5b8f11381d61a58459ffc7c595fd52d09c03c711894a63a9578043d2338412b"},"taxa":[{"id":"CWE-829","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":"c82e35ef53013361d2f294a26bac0f0edda64e76688ae16a36cf7bf55267324b"},"taxa":[{"id":"CWE-522","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-829","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":"4c41c686c7c7e6553ad5b25f80fbfdf8cf9318441f7e75b1b8214aff9fb0f5e8"},"taxa":[{"id":"CWE-829","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":"38d2995429922b1f972fa58780f28a9a4043cd9a229a166c9078fcef64334ff8"},"taxa":[{"id":"CWE-287","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":"5794fc30d6e4d5abdd71d03d96957375dc65b9191cbdef33f6df30fa3631441a"},"taxa":[{"id":"CWE-287","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":"5944285b3d73942cff2f8b886e326e1482dd7edb53a9a1aa414731a9542ff074"},"properties":{"dependency":{"package":"quinn-proto","version":"0.11.14","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","RUSTSEC-2026-0185"],"reachability":{"kind":"unknown"}}}},{"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":"6cd53f9d4e64d5e1bb0a77e598a51ff03100dcddf96b0e6f7fcb1e8564311b7e"},"properties":{"dependency":{"package":"rustls-webpki","version":"0.103.10","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA redacted]","[GHSA redacted]","RUSTSEC-2026-0098","RUSTSEC-2026-0099","RUSTSEC-2026-0104"],"reachability":{"kind":"unknown"}}}},{"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":"77acde750094f2ad5117934fb7baea55e6ce458e5808022f0df61b9162765872"},"properties":{"dependency":{"package":"pyo3","version":"0.26.0","advisory":"[GHSA redacted]","aliases":["[GHSA redacted]","RUSTSEC-2026-0176","RUSTSEC-2026-0177"],"reachability":{"kind":"unknown"}}}},{"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":"3038dd7d9a83ca9ff96c08270936df112509e56e7b56571578be7b13dac981be"},"properties":{"dependency":{"package":"pyasn1","version":"0.6.3","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA redacted]","[GHSA redacted]","PYSEC-2026-3455","PYSEC-2026-3456","PYSEC-2026-3457"],"reachability":{"kind":"not-imported"}}}},{"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":"7d2426d5480a67430a5a5164904af1f58c7597ccee6faf733bc3babe7c942932"},"properties":{"dependency":{"package":"aiohttp","version":"3.13.3","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","PYSEC-2026-2094","PYSEC-2026-2095","PYSEC-2026-2096","PYSEC-2026-2097","PYSEC-2026-2098","PYSEC-2026-2099","PYSEC-2026-2100","PYSEC-2026-2101","PYSEC-2026-2102","PYSEC-2026-2103","PYSEC-2026-2104","PYSEC-2026-2105","PYSEC-2026-2106","PYSEC-2026-2107","PYSEC-2026-2108","PYSEC-2026-2109","PYSEC-2026-2110","PYSEC-2026-2111","PYSEC-2026-2112","PYSEC-2026-2113","PYSEC-2026-237","PYSEC-2026-3545","PYSEC-2026-3546","PYSEC-2026-3547"],"reachability":{"kind":"unknown"}}}},{"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":"449b61f5c6aace9fa4fe888f6a84062bbe2d0bde8503db408b5c6f80f83669ad"},"properties":{"dependency":{"package":"urllib3","version":"2.6.3","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[GHSA redacted]","PYSEC-2026-141","PYSEC-2026-142"],"reachability":{"kind":"not-imported"}}}},{"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":"66e9e45b45176a4d36da6fcd347cb641ca1f83641601321c3106e1765c1650fa"},"properties":{"dependency":{"package":"pyjwt","version":"2.12.1","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","PYSEC-2026-175","PYSEC-2026-176","PYSEC-2026-177","PYSEC-2026-178","PYSEC-2026-179"],"reachability":{"kind":"unknown"}}}},{"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":"5043291ff7cc5d2d16df907da5debda2514aa0f0ead3b3e22b7b0c3fe1df388f"},"properties":{"dependency":{"package":"cryptography","version":"46.0.5","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[CVE redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","[GHSA redacted]","PYSEC-2026-35","PYSEC-2026-3552","PYSEC-2026-3553","PYSEC-2026-3554","PYSEC-2026-36"],"reachability":{"kind":"not-imported"}}}},{"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":"5da7522e609e0e26fd2efed1644de4bfa7d1e33a32f46a648f87c642e53503c7"},"properties":{"dependency":{"package":"thrift","version":"0.17.0","advisory":"[GHSA redacted]","aliases":["BIT-thrift-2026-43868","[CVE redacted]"],"reachability":{"kind":"unknown"}}}},{"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":"46ffea337bd3101a9b6b881134e2b16cabb8e08337706a9e2902bf1cf3b3b3bd"},"properties":{"dependency":{"package":"paste","version":"1.0.15","advisory":"RUSTSEC-2024-0436","reachability":{"kind":"unknown"}}}},{"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":"588e4d2dd962bbba32c8ad398ceb9df29ff90e8484f3d9286fb2e28fc5caf0ce"},"properties":{"dependency":{"package":"rustls-pemfile","version":"2.2.0","advisory":"RUSTSEC-2025-0134","reachability":{"kind":"unknown"}}}},{"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":"8a2bea91c5aca8ae5c923665fc15c3f2d6e88fc2f40951258a89e8f988d5e641"},"properties":{"dependency":{"package":"rand","version":"0.10.0","advisory":"RUSTSEC-2026-0097","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"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":"e2f8eb08549dd27f40d98b81beb1053addb202789aaf2a45ddd139e3062ceb24"},"properties":{"dependency":{"package":"rand","version":"0.9.2","advisory":"RUSTSEC-2026-0097","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"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":"b98ae5f30c5b6fbc6cae15d9dc5cd76a693433f42b9cacfa1d22c0255d553930"},"properties":{"dependency":{"package":"anyhow","version":"1.0.102","advisory":"RUSTSEC-2026-0190","reachability":{"kind":"unknown"}}}},{"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":"64ea38212677a97042310383f166265b9cb7f8f7d3bab8300186fbc964797142"},"properties":{"dependency":{"package":"quick-xml","version":"0.38.4","advisory":"RUSTSEC-2026-0194","aliases":["RUSTSEC-2026-0195"],"reachability":{"kind":"unknown"}}}},{"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":"dc306e2ac45194bf6b5909a0a3753bd487afc232da37052826d5ebb4f860e6a4"},"properties":{"dependency":{"package":"h2","version":"0.4.13","advisory":"RUSTSEC-2026-0258","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"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":"827d8d6d582e9366c3ed1ace07e2f5d2cd24d17e00b8a898a4ad3d0ad77dad34"},"properties":{"dependency":{"package":"rustls","version":"0.23.37","advisory":"RUSTSEC-2026-0285","aliases":["[GHSA redacted]"],"reachability":{"kind":"unknown"}}}},{"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":"835e87b62c8e40bf60d2d5b51fae9bdf31ddd3e79a3f1ad05586d01143acac23"},"properties":{"dependency":{"package":"idna","version":"3.11","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","PYSEC-2026-215"],"reachability":{"kind":"not-imported"}}}},{"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":"6c3db367eaba5269fb1b20ed9ee2ab85a753d0d303d333782461c2549a41399a"},"properties":{"dependency":{"package":"pytest","version":"9.0.2","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","PYSEC-2026-1845"],"reachability":{"kind":"test-or-tooling","file":"python/delta_sharing/tests/conftest.py","line":22}}}},{"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":"c09eb3fb28399639309912704da7e207a2eba836b389dfd449fd5a9098432de1"},"properties":{"dependency":{"package":"jwcrypto","version":"1.5.6","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","PYSEC-2026-70"],"reachability":{"kind":"production","file":"python/delta_sharing/_internal_auth.py","line":23}}}},{"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":"9f65ce96bd75cb3179daf705d7e8e5d1a58e92246b977dc91dd23ee500217afe"},"properties":{"dependency":{"package":"requests","version":"2.32.5","advisory":"[GHSA redacted]","aliases":["[CVE redacted]","PYSEC-2026-2275"],"reachability":{"kind":"production","file":"python/delta_sharing/_internal_auth.py","line":20}}}},{"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":"f95564ecc4ee3bd9894d02e75267ee5b83eafaeaf89bba4f4240591a97a742d6"},"properties":{"dependency":{"package":"setuptools","version":"82.0.1","advisory":"[GHSA redacted]","aliases":["BIT-setuptools-2026-59890","[CVE redacted]","PYSEC-2026-3447"],"reachability":{"kind":"unknown"}}}},{"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":"388b32cd0024039f8ae1eaa7941b452de69e9d2dfc5b2ca866db1b33ea2ad1fe"},"properties":{"dependency":{"package":"click","version":"8.3.1","advisory":"PYSEC-2026-2132","aliases":["[CVE redacted]","[GHSA redacted]"],"reachability":{"kind":"not-imported"}}}},{"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":"9a3ea34d241697ee470e469c74931163901918c08bb46c3cc5282e1852b62504"},"properties":{"dependency":{"package":"pygments","version":"2.19.2","advisory":"PYSEC-2026-2987","aliases":["[CVE redacted]","[GHSA redacted]"],"reachability":{"kind":"not-imported"}}}},{"ruleId":"D34","level":"note","message":{"text":"Orphaned files with no living knowledge: 29 of 65 analysed file(s) have no living knowledge left \u2014 their last meaningful change has decayed away, so if one breaks, no one currently understands it (counted over production source files of roughly 2,400 bytes or more, excluding vendored, generated and example/demo trees and test files identified by path convention, largest first; 65 of the 84 production source files in this repository met that bar). None is large enough to earn a read-through of its own, so this row stands in for the per-file rows rather than raising one each \u2014 most significant first: python/delta_sharing/_internal_auth.py, server/src/main/scala/io/delta/standalone/internal/DeltaSharingCDCReader.scala, server/src/main/scala/io/delta/sharing/server/common/actions/DeltaAction.scala, server/src/main/scala/io/delta/sharing/server/common/CloudFileSigner.scala, server/src/main/scala/io/delta/sharing/kernel/internal/PredicateConverter.scala, client/src/main/scala/io/delta/sharing/spark/RemoteDeltaCDFRelation.scala, server/src/main/scala/io/delta/sharing/server/SharedTableManager.scala, server/src/main/scala/io/delta/sharing/kernel/internal/JsonPredicatePruner.scala (and 21 more). Attach the read to the next change that touches one of them: have a second person review that change, and leave behind a short comment or test recording what the file is for, so the knowledge comes back at the cost of a change you were making anyway."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ce861fd78f6935114d3a342cb90ad25870f984e1042954fcbbe27c0e23be3658"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: DeltaSharedTableKernel.scala \u2194 DeltaSharingService.scala: \u0060server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala\u0060 and \u0060server/src/main/scala/io/delta/sharing/server/DeltaSharingService.scala\u0060 change together 77% of the time (10 of the 13 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well \u2014 a repo-wide or module-wide sweep is evidence about the sweep rather than about any pair inside it and is left out of BOTH sides of this ratio, while a dependency bump, a formatter/rename sweep, or a commit whose edit to one of the two files was a tool directive such as //go:generate or whitespace only is left out of the shared count ONLY, so the two sides are not taken over identical commit sets) with no explicit dependency \u2014 the edge is real but nothing declares it. Read the pair before acting: if one registers itself into the other through a hook or an initialiser, the missing dependency is DELIBERATE \u2014 the registration is the link, and it is meant not to be an import \u2014 and the thing to add is a comment on each side naming the other, not a merge; if they simply belong together, co-locate them; if neither holds, the coupling is hidden and worth breaking. You can check this without leaving the row: of the 10 shared commits counted here, the most recent 3 are \u0060991ed0d7\u0060 OSS server: support fileidhash on queryTable and queryCDF (#856); \u0060dd423305\u0060 Support includeEndStreamAction for cdf queries in both server and cli\u2026; \u0060ec4733ad\u0060 Support includeEndStreamAction for listFiles in server (#577) \u2014 run \u0060git show\u0060 on any of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/src/main/scala/io/delta/sharing/kernel/internal/DeltaSharedTableKernel.scala"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2b3dc5872c39fb373f87853a76d696aebff0eb2037e55e79ff25eccc7169299e"}},{"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":"M1","level":"warning","message":{"text":"README may be stale: 26 code files changed in the last 6 months but the README was not touched \u2014 it may no longer reflect the system."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e973e7a8d5d866995cbf78b120250fe7f132bf928812d48ca62e69aba11b9910"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: Apache Spark connector and server are present but the README omits the Spark warehouse project \u2014 reported by the model that read the README against this repository; no term search was run for this one, so nothing here has been checked against the tree. Treat it as a reading to confirm, not as a measured contradiction: verify it against the code before acting on it, and if the footprint it describes does exist, this row is wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"e9282efca67476c76286f109b07c56bb7daa901957d6246a60362de64be8b65d"}},{"ruleId":"M4","level":"note","message":{"text":"README/code drift: README advertises Docker containerisation, but no Dockerfile/compose file exists \u2014 searched for: \u0060dockerfile\u0060, \u0060docker-compose\u0060, \u0060compose.yaml\u0060, \u0060compose.yml\u0060. Each was matched case- and separator-insensitively against file and directory NAMES anywhere in the tree, and against the CONTENTS of manifest files (package.json, *.csproj, *.props, *.slnx, *.yml, Dockerfile); the README\u0027s own prose never counts, so a claim is never refuted by merely being made. Nothing outside that search was read \u2014 a footprint living only in a submodule, in a file type not listed here, or under a name none of those terms matches is not seen, and this row is then wrong."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"d116e696b89ee5b5606de54018788ae795afe8387ffcf59a3c5f5c024f4aec80"}}],"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-287","guid":"ec161954-9db0-355d-9539-ae741453fd44","name":"CWE-287","shortDescription":{"text":"CWE-287"},"helpUri":"https://cwe.mitre.org/data/definitions/287.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-506","guid":"401d6455-56e3-0552-9a39-f77461673e3f","name":"CWE-506","shortDescription":{"text":"CWE-506"},"helpUri":"https://cwe.mitre.org/data/definitions/506.html"},{"id":"CWE-522","guid":"71fb233e-ce6a-ae57-9419-ef8373540b09","name":"CWE-522","shortDescription":{"text":"CWE-522"},"helpUri":"https://cwe.mitre.org/data/definitions/522.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-829","guid":"13c33925-97fb-5a5e-b40c-56d328b8a4d7","name":"CWE-829","shortDescription":{"text":"CWE-829"},"helpUri":"https://cwe.mitre.org/data/definitions/829.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":38,"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."]}]}