[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16676837#comment-16676837
]
Sean Busbey commented on HBASE-21247:
-
Also Ted please update the subject to match the problem that
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16676833#comment-16676833
]
Sean Busbey commented on HBASE-21247:
-
Yes, it should go to all branches that got HBASE-20856
>
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16676602#comment-16676602
]
Hudson commented on HBASE-21247:
Results for branch master
[build #588 on
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16675963#comment-16675963
]
Duo Zhang commented on HBASE-21247:
---
If this is a bug then we should push it to all related branches?
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16675358#comment-16675358
]
Sean Busbey commented on HBASE-21247:
-
+1
nit:
{code}
129 public Class
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16668811#comment-16668811
]
Josh Elser commented on HBASE-21247:
I think I saw Stack saw that Busbey was traveling at the
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16667748#comment-16667748
]
Ted Yu commented on HBASE-21247:
Gentle ping [~busbey]
> Custom WAL Provider cannot be specified by
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16662407#comment-16662407
]
Ted Yu commented on HBASE-21247:
[~busbey]:
Mind taking another look ?
> Custom WAL Provider cannot be
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659674#comment-16659674
]
Hadoop QA commented on HBASE-21247:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659260#comment-16659260
]
Hadoop QA commented on HBASE-21247:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659222#comment-16659222
]
Sean Busbey commented on HBASE-21247:
-
probably best to just save a copy of the class name for
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659221#comment-16659221
]
Sean Busbey commented on HBASE-21247:
-
oh even better, WALFactory is the one that does the wrapping
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659218#comment-16659218
]
Sean Busbey commented on HBASE-21247:
-
we also probably don't want to use the
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659176#comment-16659176
]
Ted Yu commented on HBASE-21247:
Patch v10 would show the exception for testCustomProvider.
> Custom
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659175#comment-16659175
]
Ted Yu commented on HBASE-21247:
If we go the above route, there is another issue we need to solve.
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659153#comment-16659153
]
Sean Busbey commented on HBASE-21247:
-
Catching a {{Throwable}} is v dangerous and IMHO not
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16659071#comment-16659071
]
Ted Yu commented on HBASE-21247:
Since the added logic is for meta WAL Provider, how about moving that
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16658950#comment-16658950
]
Sean Busbey commented on HBASE-21247:
-
okay I've made a first pass. I like the added test and the
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16656797#comment-16656797
]
Sean Busbey commented on HBASE-21247:
-
FYI Ted, I have this on my list of things to do today, but it
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16650476#comment-16650476
]
Sean Busbey commented on HBASE-21247:
-
Reviews take time Ted and you posted v8 on Friday. Please be
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16650362#comment-16650362
]
Ted Yu commented on HBASE-21247:
[~busbey]:
Can you take a look at patch v8 ?
> Custom WAL Provider
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648630#comment-16648630
]
Hadoop QA commented on HBASE-21247:
---
| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote ||
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648372#comment-16648372
]
Ted Yu commented on HBASE-21247:
See if patch v8 is better.
Class retrieval from ordinary WAL provider
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648186#comment-16648186
]
Sean Busbey commented on HBASE-21247:
-
bq. One more thing, use isAssignableFrom(..) to make sure the
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648182#comment-16648182
]
Sean Busbey commented on HBASE-21247:
-
bq. What about using the set of enum values to determine if
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648137#comment-16648137
]
Josh Elser commented on HBASE-21247:
Also, please add DEBUG logging for the case when we fail to
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16648133#comment-16648133
]
Josh Elser commented on HBASE-21247:
{code:java}
+String clsName = conf.get(key, defaultValue);
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16647993#comment-16647993
]
Ted Yu commented on HBASE-21247:
bq. in v6 you still have two different places that try to load by
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16647917#comment-16647917
]
Sean Busbey commented on HBASE-21247:
-
in v6 you still have two different places that try to load by
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16647910#comment-16647910
]
Ted Yu commented on HBASE-21247:
The test failures for patch v6 were not related to the patch.
{code}
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16647880#comment-16647880
]
Hudson commented on HBASE-21247:
Results for branch master
[build #541 on
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16647680#comment-16647680
]
Hadoop QA commented on HBASE-21247:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16647351#comment-16647351
]
Ted Yu commented on HBASE-21247:
For meta WAL provider, if the user wants to use the same one for
[
https://issues.apache.org/jira/browse/HBASE-21247?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16647312#comment-16647312
]
Hadoop QA commented on HBASE-21247:
---
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem
34 matches
Mail list logo