[
https://issues.apache.org/jira/browse/HDDS-8268?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Siyao Meng updated HDDS-8268:
-----------------------------
Description:
We could try to enable some [spotbugs|https://spotbugs.github.io/] that could
have caught a bunch of not-closed handles before a commit is merged.
For one,
bq. OBL: Method may fail to clean up stream or resource on checked exception
(OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE)
https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#obl-method-may-fail-to-clean-up-stream-or-resource-on-checked-exception-obl-unsatisfied-obligation-exception-edge
which could have discovered some of the leaking handle issue before a commit is
merged.
Not sure about the false-positive rate though. Have to try to find out.
Just recently we have quite a few PRs just to fix the leaking handles:
1. HDDS-8251. RocksIterator is not closed properly in snapshot
2. HDDS-8265. ManagedColumnFamilyOptions is not closed properly in
OmSnapshotManager
3. HDDS-8163. Use try-with-resources to ensure close rockdb connection in
SstFilteringService
4. HDDS-7958. Ozone client not closed in integration tests
5. HDDS-8145. ReadReplicas should close client
was:
We could try to enable some [spotbugs|https://spotbugs.github.io/] that could
have caught a bunch of not-closed handles before a commit is merged.
For one,
bq. OBL: Method may fail to clean up stream or resource on checked exception
(OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE)
https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#obl-method-may-fail-to-clean-up-stream-or-resource-on-checked-exception-obl-unsatisfied-obligation-exception-edge
which could have discovered some of the leaking handle issue before a commit is
merged.
Just recently we have quite a few PRs just to fix the leaking handles:
1. HDDS-8251. RocksIterator is not closed properly in snapshot
2. HDDS-8265. ManagedColumnFamilyOptions is not closed properly in
OmSnapshotManager
3. HDDS-8163. Use try-with-resources to ensure close rockdb connection in
SstFilteringService
4. HDDS-7958. Ozone client not closed in integration tests
5. HDDS-8145. ReadReplicas should close client
> spotbugs: enable stream or resource clean up check
> --------------------------------------------------
>
> Key: HDDS-8268
> URL: https://issues.apache.org/jira/browse/HDDS-8268
> Project: Apache Ozone
> Issue Type: Wish
> Components: CI
> Reporter: Siyao Meng
> Priority: Major
>
> We could try to enable some [spotbugs|https://spotbugs.github.io/] that could
> have caught a bunch of not-closed handles before a commit is merged.
> For one,
> bq. OBL: Method may fail to clean up stream or resource on checked exception
> (OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE)
> https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#obl-method-may-fail-to-clean-up-stream-or-resource-on-checked-exception-obl-unsatisfied-obligation-exception-edge
> which could have discovered some of the leaking handle issue before a commit
> is merged.
> Not sure about the false-positive rate though. Have to try to find out.
> Just recently we have quite a few PRs just to fix the leaking handles:
> 1. HDDS-8251. RocksIterator is not closed properly in snapshot
> 2. HDDS-8265. ManagedColumnFamilyOptions is not closed properly in
> OmSnapshotManager
> 3. HDDS-8163. Use try-with-resources to ensure close rockdb connection in
> SstFilteringService
> 4. HDDS-7958. Ozone client not closed in integration tests
> 5. HDDS-8145. ReadReplicas should close client
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
