Umeshkumar9414 commented on code in PR #6129:
URL: https://github.com/apache/hbase/pull/6129#discussion_r1717533112
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/RSProcedureDispatcher.java:
##########
@@ -471,9 +474,10 @@ public ServerOperation(RemoteProcedure remoteProcedure,
long procId, Class<?> rs
this.rsProcData = rsProcData;
}
- public RemoteProcedureRequest buildRequest() {
+ public RemoteProcedureRequest buildRequest(long
initiatingMasterActiveTime) {
Review Comment:
There were other reasons also like there is a difference in how we create
requests for OpenRegion and CloseRegions. For OpenRegionRequest we create only
one request for all the OpenRegionOperations but for close we create
CloseRegoinRequest for each operation.
It would be all good in close but for OpenRegionRequest
[here](https://github.com/apache/hbase/pull/6129/files#diff-f8d7389a8459b81b822e8ebbc129189771c1e203ac9268e3ee003d8131de9343R447),
we need to get the activeCode from one of the entry.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
