[
https://issues.apache.org/jira/browse/HBASE-18884?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16185175#comment-16185175
]
Hudson commented on HBASE-18884:
--------------------------------
SUCCESS: Integrated in Jenkins build HBase-Trunk_matrix #3795 (See
[https://builds.apache.org/job/HBase-Trunk_matrix/3795/])
HBASE-18884 Coprocessor Design Improvements follow up of HBASE-17732 (appy: rev
74d0adce61fc39ef9d1ec2254dcd0f53181bb62c)
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/BaseRowProcessorEndpoint.java
* (edit)
hbase-examples/src/main/java/org/apache/hadoop/hbase/coprocessor/example/RefreshHFilesEndpoint.java
* (edit)
hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupAdminEndpoint.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/TokenProvider.java
* (edit)
hbase-endpoint/src/test/java/org/apache/hadoop/hbase/coprocessor/ColumnAggregationEndpoint.java
* (edit)
hbase-endpoint/src/test/java/org/apache/hadoop/hbase/coprocessor/TestRegionServerCoprocessorEndpoint.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/CoprocessorServiceBackwardCompatiblity.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/VisibilityController.java
* (edit) hbase-client/src/main/java/org/apache/hadoop/hbase/Coprocessor.java
* (edit)
hbase-endpoint/src/main/java/org/apache/hadoop/hbase/coprocessor/AggregateImplementation.java
* (edit)
hbase-endpoint/src/main/java/org/apache/hadoop/hbase/security/access/SecureBulkLoadEndpoint.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/WALCoprocessor.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionCoprocessorHost.java
* (edit)
hbase-endpoint/src/test/java/org/apache/hadoop/hbase/coprocessor/ProtobufCoprocessorService.java
* (add)
dev-support/design-docs/Coprocessor_Design_Improvements-Use_composition_instead_of_inheritance-HBASE-17732-2017_09_27.pdf
* (edit)
hbase-server/src/test/java/org/apache/hadoop/hbase/security/access/TestAccessController.java
* (edit)
hbase-examples/src/main/java/org/apache/hadoop/hbase/coprocessor/example/RowCountEndpoint.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionServerCoprocessorHost.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
* (edit)
hbase-endpoint/src/test/java/org/apache/hadoop/hbase/coprocessor/ColumnAggregationEndpointWithErrors.java
* (edit)
hbase-endpoint/src/main/java/org/apache/hadoop/hbase/coprocessor/Export.java
* (edit)
hbase-endpoint/src/test/java/org/apache/hadoop/hbase/regionserver/TestServerCustomProtocol.java
* (edit)
hbase-endpoint/src/test/java/org/apache/hadoop/hbase/coprocessor/ColumnAggregationEndpointNullResponse.java
* (edit)
hbase-endpoint/src/test/java/org/apache/hadoop/hbase/coprocessor/TestAsyncCoprocessorEndpoint.java
* (edit)
hbase-examples/src/main/java/org/apache/hadoop/hbase/coprocessor/example/BulkDeleteEndpoint.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/MultiRowMutationEndpoint.java
* (edit)
hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterCoprocessorHost.java
HBASE-18884 Coprocessor Design Improvements follow up of HBASE-17732; (stack:
rev b5b4108fcedeea4e25e86f426e313ffa51229d10)
* (add) dev-support/design-docs/README.txt
* (edit) hbase-client/src/main/java/org/apache/hadoop/hbase/Coprocessor.java
> Coprocessor Design Improvements follow up of HBASE-17732
> --------------------------------------------------------
>
> Key: HBASE-18884
> URL: https://issues.apache.org/jira/browse/HBASE-18884
> Project: HBase
> Issue Type: Improvement
> Components: Coprocessors
> Reporter: Appy
> Assignee: Appy
> Labels: incompatible
> Fix For: 2.0.0-alpha-4
>
> Attachments: HBASE-18884.master.001.patch
>
>
> Creating new jira to track suggestions that came in review
> (https://reviews.apache.org/r/62141/) but are not blocker and can be done
> separately.
> Suggestions by [~apurtell]
> - Change {{Service Coprocessor#getService()}} to {{List<Service>
> Coprocessor#getServices()}}
> - I think we overstepped by offering [table resource management via this
> interface|https://github.com/apache/hbase/blob/master/hbase-client/src/main/java/org/apache/hadoop/hbase/CoprocessorEnvironment.java#L57].
> There are a lot of other internal resource types which could/should be
> managed this way but they are all left up to the implementor. Perhaps we
> should remove the table ref management and leave it up to them as well.
> ----
> - Checkin the finalized design doc into repo
> (https://docs.google.com/document/d/1mPkM1CRRvBMZL4dBQzrus8obyvNnHhR5it2yyhiFXTg/edit)
> (fyi: [~stack])
> - Added example to javadoc of Coprocessor base interface on how to implement
> one in the new design
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
