[ https://issues.apache.org/jira/browse/HBASE-5070?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13173466#comment-13173466 ]
jirapos...@reviews.apache.org commented on HBASE-5070: ------------------------------------------------------ ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3273/#review4022 ----------------------------------------------------------- src/docbkx/book.xml <https://reviews.apache.org/r/3273/#comment9121> Should read 'checking is enabled' src/docbkx/book.xml <https://reviews.apache.org/r/3273/#comment9122> When would the URL be active ? It is not available now. src/main/java/org/apache/hadoop/hbase/constraint/Constraint.java <https://reviews.apache.org/r/3273/#comment9123> Whitespace should be removed. src/main/java/org/apache/hadoop/hbase/constraint/Constraints.java <https://reviews.apache.org/r/3273/#comment9124> I think this should start with 'Constraint Class ' - Ted On 2011-12-20 19:14:46, Jesse Yates wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/3273/ bq. ----------------------------------------------------------- bq. bq. (Updated 2011-12-20 19:14:46) bq. bq. bq. Review request for hbase, Gary Helmling, Ted Yu, and Michael Stack. bq. bq. bq. Summary bq. ------- bq. bq. Follow-up on changes to constraint as per stack's comments on HBASE-4605. bq. bq. bq. This addresses bug HBASE-5070. bq. https://issues.apache.org/jira/browse/HBASE-5070 bq. bq. bq. Diffs bq. ----- bq. bq. src/docbkx/book.xml bd3f881 bq. src/main/java/org/apache/hadoop/hbase/constraint/BaseConstraint.java 7ce6d45 bq. src/main/java/org/apache/hadoop/hbase/constraint/Constraint.java 2d8b4d7 bq. src/main/java/org/apache/hadoop/hbase/constraint/Constraints.java 7825466 bq. src/main/java/org/apache/hadoop/hbase/constraint/package-info.java 6145ed5 bq. src/test/java/org/apache/hadoop/hbase/constraint/CheckConfigurationConstraint.java c49098d bq. bq. Diff: https://reviews.apache.org/r/3273/diff bq. bq. bq. Testing bq. ------- bq. bq. mvn clean test -P localTests -Dest=*Constraint* <- all tests pass. bq. bq. bq. Thanks, bq. bq. Jesse bq. bq. > Constraints implementation and javadoc changes > ---------------------------------------------- > > Key: HBASE-5070 > URL: https://issues.apache.org/jira/browse/HBASE-5070 > Project: HBase > Issue Type: Task > Reporter: Zhihong Yu > > This is continuation of HBASE-4605 > See Stack's comments https://reviews.apache.org/r/2579/#review3980 -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira