You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by st...@duboce.net on 2010/05/26 00:54:43 UTC
Review Request: Patch for 0.20.5 inclusion
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/88/
-----------------------------------------------------------
Review request for hbase.
Summary
-------
Version of hbase-2599 for 0.20 branch for 0.20.5.
It cuts DNS lookups from HServerInfo and uses the regionservers name everywhere instead of ip sometimes and a lookedup hostname at other times. Also puts hostname:port into .META. rather than IP.
This addresses bug hbase-2599.
Diffs
-----
branches/0.20/src/java/org/apache/hadoop/hbase/ClusterStatus.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/HServerInfo.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/BaseScanner.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/HMaster.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 948218
branches/0.20/src/test/org/apache/hadoop/hbase/TestServerInfo.java PRE-CREATION
branches/0.20/src/webapps/master/table.jsp 948218
Diff: http://review.hbase.org/r/88/diff
Testing
-------
Doing now.
Thanks,
stack
Re: Review Request: Patch for hbase-2599 for 0.20.5 inclusion
Posted by Ryan Rawson <ry...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/88/#review75
-----------------------------------------------------------
Ship it!
+1 lets do it
- Ryan
On 2010-05-25 16:04:24, stack wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://review.hbase.org/r/88/
> -----------------------------------------------------------
>
> (Updated 2010-05-25 16:04:24)
>
>
> Review request for hbase.
>
>
> Summary
> -------
>
> Version of hbase-2599 for 0.20 branch for 0.20.5.
>
> It cuts DNS lookups from HServerInfo and uses the regionservers name everywhere instead of ip sometimes and a lookedup hostname at other times. Also puts hostname:port into .META. rather than IP. Removing DNS lookups makes it so there is no possibility for disagreement over hostname if master gets one answer for an IP and the regionserver has another.
>
> Other notes:
>
> It replaces variable servername with hostnameandport so its clear whats going on.
>
> Does some cleanup in HServerInfo renaming data member 'name' as 'hostname' and then purges DNS caching and lookups from this class. It removes some unnecessary setters and does some javadoc fix ups.
>
> Removes the dumb serverAddressToServerInfo from ServerManager.
>
>
> This addresses bug hbase-2599.
>
>
> Diffs
> -----
>
> branches/0.20/src/java/org/apache/hadoop/hbase/ClusterStatus.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/HServerInfo.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/BaseScanner.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/HMaster.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 948218
> branches/0.20/src/test/org/apache/hadoop/hbase/TestServerInfo.java PRE-CREATION
> branches/0.20/src/webapps/master/table.jsp 948218
>
> Diff: http://review.hbase.org/r/88/diff
>
>
> Testing
> -------
>
> Doing now.
>
>
> Thanks,
>
> stack
>
>
Re: Review Request: Patch for hbase-2599 for 0.20.5 inclusion
Posted by Todd Lipcon <to...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/88/#review81
-----------------------------------------------------------
Ship it!
this seems reasonable, but only way we'll know is testing on various systems with different kinds of DNS setups. Can we get those who have had issues to try this and make sure it fixes, before we release it?
branches/0.20/src/java/org/apache/hadoop/hbase/ClusterStatus.java
<http://review.hbase.org/r/88/#comment409>
I don't like that this function is called getServerNames and returns hostnameports.
branches/0.20/src/test/org/apache/hadoop/hbase/TestServerInfo.java
<http://review.hbase.org/r/88/#comment410>
using the junit4 expected thing here doesn't quite work, cuz we don't know that it didn't throw on line 34, right?
- Todd
On 2010-05-25 16:04:24, stack wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://review.hbase.org/r/88/
> -----------------------------------------------------------
>
> (Updated 2010-05-25 16:04:24)
>
>
> Review request for hbase.
>
>
> Summary
> -------
>
> Version of hbase-2599 for 0.20 branch for 0.20.5.
>
> It cuts DNS lookups from HServerInfo and uses the regionservers name everywhere instead of ip sometimes and a lookedup hostname at other times. Also puts hostname:port into .META. rather than IP. Removing DNS lookups makes it so there is no possibility for disagreement over hostname if master gets one answer for an IP and the regionserver has another.
>
> Other notes:
>
> It replaces variable servername with hostnameandport so its clear whats going on.
>
> Does some cleanup in HServerInfo renaming data member 'name' as 'hostname' and then purges DNS caching and lookups from this class. It removes some unnecessary setters and does some javadoc fix ups.
>
> Removes the dumb serverAddressToServerInfo from ServerManager.
>
>
> This addresses bug hbase-2599.
>
>
> Diffs
> -----
>
> branches/0.20/src/java/org/apache/hadoop/hbase/ClusterStatus.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/HServerInfo.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/BaseScanner.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/HMaster.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java 948218
> branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 948218
> branches/0.20/src/test/org/apache/hadoop/hbase/TestServerInfo.java PRE-CREATION
> branches/0.20/src/webapps/master/table.jsp 948218
>
> Diff: http://review.hbase.org/r/88/diff
>
>
> Testing
> -------
>
> Doing now.
>
>
> Thanks,
>
> stack
>
>
Re: Review Request: Patch for hbase-2599 for 0.20.5 inclusion
Posted by st...@duboce.net.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/88/
-----------------------------------------------------------
(Updated 2010-05-25 16:04:24.285624)
Review request for hbase.
Changes
-------
Added some description
Summary (updated)
-------
Version of hbase-2599 for 0.20 branch for 0.20.5.
It cuts DNS lookups from HServerInfo and uses the regionservers name everywhere instead of ip sometimes and a lookedup hostname at other times. Also puts hostname:port into .META. rather than IP. Removing DNS lookups makes it so there is no possibility for disagreement over hostname if master gets one answer for an IP and the regionserver has another.
Other notes:
It replaces variable servername with hostnameandport so its clear whats going on.
Does some cleanup in HServerInfo renaming data member 'name' as 'hostname' and then purges DNS caching and lookups from this class. It removes some unnecessary setters and does some javadoc fix ups.
Removes the dumb serverAddressToServerInfo from ServerManager.
This addresses bug hbase-2599.
Diffs
-----
branches/0.20/src/java/org/apache/hadoop/hbase/ClusterStatus.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/HServerInfo.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/BaseScanner.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/HMaster.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 948218
branches/0.20/src/test/org/apache/hadoop/hbase/TestServerInfo.java PRE-CREATION
branches/0.20/src/webapps/master/table.jsp 948218
Diff: http://review.hbase.org/r/88/diff
Testing
-------
Doing now.
Thanks,
stack
Re: Review Request: Patch for hbase-2599 for 0.20.5 inclusion
Posted by st...@duboce.net.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.hbase.org/r/88/
-----------------------------------------------------------
(Updated 2010-05-25 15:55:50.626407)
Review request for hbase.
Changes
-------
Changed summary
Summary (updated)
-------
Version of hbase-2599 for 0.20 branch for 0.20.5.
It cuts DNS lookups from HServerInfo and uses the regionservers name everywhere instead of ip sometimes and a lookedup hostname at other times. Also puts hostname:port into .META. rather than IP.
This addresses bug hbase-2599.
Diffs
-----
branches/0.20/src/java/org/apache/hadoop/hbase/ClusterStatus.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/HServerInfo.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/BaseScanner.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/HMaster.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/ProcessRegionOpen.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/master/ServerManager.java 948218
branches/0.20/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java 948218
branches/0.20/src/test/org/apache/hadoop/hbase/TestServerInfo.java PRE-CREATION
branches/0.20/src/webapps/master/table.jsp 948218
Diff: http://review.hbase.org/r/88/diff
Testing
-------
Doing now.
Thanks,
stack