[
https://issues.apache.org/jira/browse/HDDS-332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16590834#comment-16590834
]
Dinesh Chitlangia commented on HDDS-332:
----------------------------------------
[~anu] Thank you for working on this.
Could you please take a look at following:
1. StorageHandlerBuilder.getStorageHandler()
{{- Configuration conf = new OzoneConfiguration(); }}
{{- return new LocalStorageHandler(conf); }}
{{+ LOG.error("No Storage Handler Configured."); }}
{{+ throw new IOException("Invalid Handler Configuration");}}
I understand the purpose of adding these lines in the else block in case
storageHandler is null. However, since we specify distributed as the default
value for storageHandler, I am wondering if we will run into a scenario where
storageHandler is null. Is there some additional check that we must do
elsewhere to have the default value set correctly?
2. Some test classes specify OZONE_HANDLER_TYPE_KEY = "local" in comments. Do
you think it will be a good idea to see if those comments should be updated
too, to avoid any ambiguity?
Following test classes can be checked:
TestVolume.java
TestBuckets.java
TestOzoneWebAccess.java
TestOzoneLocalVolumes.java
> Remove the ability to configure ozone.handler.type
> --------------------------------------------------
>
> Key: HDDS-332
> URL: https://issues.apache.org/jira/browse/HDDS-332
> Project: Hadoop Distributed Data Store
> Issue Type: Bug
> Reporter: Dinesh Chitlangia
> Assignee: Dinesh Chitlangia
> Priority: Major
> Fix For: 0.2.1
>
> Attachments: HDDS-332.001.patch
>
>
> Based on feedback from [~anu] and [~xyao], logging this Jira to remove the
> ability to configure ozone.handler.type.
> Currently, it can be configured with values - distributed or local.
> Value 'local' was strictly for internal testing purposes in initial stages.
> By removing the configurability, we can avoid situations where users may end
> up misconfiguring and running into issues.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
