junegunn opened a new pull request, #6986: URL: https://github.com/apache/hbase/pull/6986
HBASE-29137 was backported to branch-2 in #6799, but @guluo2016 pointed out that it doesn't perform verification of the column-familiy level configuration. * See https://github.com/apache/hbase/pull/6799#discussion_r2084374632 This was due to the change of the alter command of HBase shell in #1959 in master branch which decided to use `setValue` instead of `setConfiguration`. So in branch-2, we need to check the values in the `configuration`, instead of `values`. But in this patch, I updated the code to check both. The unit test was unable to detect this because it was using `setValue` instead of `setConfiguration`. @Apache9 Do you think we also need this in master branch? Could you explain the difference between "values" and "configuration"? Are we phasing out the latter? -- 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]
