Apache9 commented on a change in pull request #3497:
URL: https://github.com/apache/hbase/pull/3497#discussion_r671014258
##########
File path: hbase-protocol-shaded/src/main/protobuf/client/Filter.proto
##########
@@ -88,10 +88,6 @@ message FilterWrapper {
message FirstKeyOnlyFilter {
}
-message FirstKeyValueMatchingQualifiersFilter {
Review comment:
Oh, filter has protos...
Suggest sending an email to the dev list to ask if it is OK to remove the
proto defination.
The problem here is that, we guarantee old client could communicate with new
server, so here if we remove this proto, old client can not communicate with
new server any more if it make use of this filter?
--
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]
