hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Gray (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HBASE-2597) Use "Builder" pattern instead of many-argument constructors
Date Fri, 21 May 2010 23:32:16 GMT

    [ https://issues.apache.org/jira/browse/HBASE-2597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12870227#action_12870227
] 

Jonathan Gray commented on HBASE-2597:
--------------------------------------

agree that HTD and HCD could use come improvement.  Unsure about SF.  Shouldn't be many of
these bigger changes like what blooms had to introduce though I feel your pain.

> Use "Builder" pattern instead of many-argument constructors
> -----------------------------------------------------------
>
>                 Key: HBASE-2597
>                 URL: https://issues.apache.org/jira/browse/HBASE-2597
>             Project: Hadoop HBase
>          Issue Type: Improvement
>          Components: client
>            Reporter: Todd Lipcon
>            Assignee: Todd Lipcon
>             Fix For: 0.21.0
>
>
> We've got a lot of objects that have a ton of different constructors with a huge number
of parameters. Whenever we add a new parameter, existing callers break, and it's sometimes
difficult to keep track of which booleans/nulls correspond to which parameter.
> I'd like to consider moving to the "Builder" pattern in some of these cases. See http://guava-libraries.googlecode.com/svn/trunk/javadoc/com/google/common/collect/MapMaker.html
for an example of this pattern in action. Another good example is the builder API generated
by protocol buffers (search for "builder" on http://code.google.com/apis/protocolbuffers/docs/javatutorial.html
)
> I think this pattern makes code more readable and also allows us to more easily change
around the number of arguments in our constructors.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message