saintstack commented on a change in pull request #173: HBASE-22277 Removed deprecated methods from Get URL: https://github.com/apache/hbase/pull/173#discussion_r277515627
########## File path: hbase-client/src/main/java/org/apache/hadoop/hbase/client/Get.java ########## @@ -260,30 +228,6 @@ public Get setTimestamp(long timestamp) { return (Get) super.setColumnFamilyTimeRange(cf, minStamp, maxStamp); } - /** - * Get all available versions. - * @return this for invocation chaining - * @deprecated It is easy to misunderstand with column family's max versions, so use - * {@link #readAllVersions()} instead. - */ - @Deprecated - public Get setMaxVersions() { - return readAllVersions(); - } - - /** - * Get up to the specified number of versions of each column. - * @param maxVersions maximum versions for each column - * @throws IOException if invalid number of versions - * @return this for invocation chaining - * @deprecated It is easy to misunderstand with column family's max versions, so use Review comment: For sure deprecated since 2.0.0? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services