[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stack updated HBASE-21071: -- Resolution: Fixed Hadoop Flags: Reviewed Fix Version/s: 2.2.0 3.0.0 Release Note: Cleanup all the cluster start override combos in HBaseTestingUtility by adding a StartMiniClusterOption and Builder. Status: Resolved (was: Patch Available) Pushed to branch-2 and to master. Thanks for the nice cleanup [~liuml07] > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Improvement > Components: test >Affects Versions: 3.0.0, 2.2.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Labels: reviewed > Fix For: 3.0.0, 2.2.0 > > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Labels: reviewed (was: ) > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Improvement > Components: test >Affects Versions: 3.0.0, 2.2.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Labels: reviewed > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.006.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Improvement > Components: test >Affects Versions: 3.0.0, 2.2.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Issue Type: Improvement (was: Bug) > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Improvement > Components: test >Affects Versions: 3.0.0, 2.2.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Affects Version/s: 2.2.0 > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0, 2.2.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stack updated HBASE-21071: -- Attachment: HBASE-21071.006.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stack updated HBASE-21071: -- Attachment: HBASE-21071.006.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.006.patch, HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stack updated HBASE-21071: -- Attachment: HBASE-21071.006.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.006.patch, HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] stack updated HBASE-21071: -- Attachment: HBASE-21071.006.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.006.patch, > HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.branch-2.006.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.006.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch, HBASE-21071.006.patch, HBASE-21071.branch-2.006.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.005.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch, > HBASE-21071.005.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.004.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch, HBASE-21071.004.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: (was: HBASE-21071-part1.004.patch) > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071-part1.004.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.003.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch, HBASE-21071.003.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.002.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch, > HBASE-21071.002.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.001.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch, HBASE-21071.001.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Status: Patch Available (was: Open) > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Updated] (HBASE-21071) HBaseTestingUtility::startMiniCluster() to use builder pattern
[ https://issues.apache.org/jira/browse/HBASE-21071?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mingliang Liu updated HBASE-21071: -- Attachment: HBASE-21071.000.patch > HBaseTestingUtility::startMiniCluster() to use builder pattern > -- > > Key: HBASE-21071 > URL: https://issues.apache.org/jira/browse/HBASE-21071 > Project: HBase > Issue Type: Bug > Components: test >Affects Versions: 3.0.0 >Reporter: Mingliang Liu >Assignee: Mingliang Liu >Priority: Major > Attachments: HBASE-21071.000.patch > > > Currently there are 13 {{startMiniCluster()}} methods to set up a mini > cluster. I'm not surprised if we have a few more in future. It's good to > support different combination of optional parameters. We have to pick up one > of them carefully while still wondering the default values of other > parameters; if we add a new option, we may bring more new methods. > One solution is to use builder pattern: create a class {{MiniClusterOptions}} > along with a static class {{MiniClusterOptionsBuilder}}, create a new method > {{startMiniCluster(MiniClusterOptions)}}. In {{master}} we delete the old 13 > methods while in branch-2, we deprecate the old 13 methods. > Thoughts? -- This message was sent by Atlassian JIRA (v7.6.3#76005)