HBASE-22755 Removed deprecated methods from Mutation
Signed-off-by: stack <stack@apache.org>
This commit is contained in:
parent
74fb2040ea
commit
808f051b8a
|
@ -301,19 +301,6 @@ public abstract class Mutation extends OperationWithAttributes implements Row, C
|
|||
return this.familyMap;
|
||||
}
|
||||
|
||||
/**
|
||||
* Method for setting the mutation's familyMap
|
||||
* @deprecated As of release 2.0.0, this will be removed in HBase 3.0.0.
|
||||
* Use {@link Mutation#Mutation(byte[], long, NavigableMap)} instead
|
||||
*/
|
||||
@Deprecated
|
||||
public Mutation setFamilyCellMap(NavigableMap<byte [], List<Cell>> map) {
|
||||
// TODO: Shut this down or move it up to be a Constructor. Get new object rather than change
|
||||
// this internal data member.
|
||||
this.familyMap = map;
|
||||
return this;
|
||||
}
|
||||
|
||||
/**
|
||||
* Method to check if the familyMap is empty
|
||||
* @return true if empty, false otherwise
|
||||
|
@ -341,17 +328,6 @@ public abstract class Mutation extends OperationWithAttributes implements Row, C
|
|||
return Bytes.compareTo(this.getRow(), d.getRow());
|
||||
}
|
||||
|
||||
/**
|
||||
* Method for retrieving the timestamp
|
||||
* @return timestamp
|
||||
* @deprecated As of release 2.0.0, this will be removed in HBase 3.0.0.
|
||||
* Use {@link #getTimestamp()} instead
|
||||
*/
|
||||
@Deprecated
|
||||
public long getTimeStamp() {
|
||||
return this.getTimestamp();
|
||||
}
|
||||
|
||||
/**
|
||||
* Method for retrieving the timestamp.
|
||||
*
|
||||
|
|
|
@ -249,17 +249,6 @@ public class Put extends Mutation implements HeapSize {
|
|||
return (Put) super.setDurability(d);
|
||||
}
|
||||
|
||||
/**
|
||||
* Method for setting the put's familyMap
|
||||
* @deprecated As of release 2.0.0, this will be removed in HBase 3.0.0.
|
||||
* Use {@link Put#Put(byte[], long, NavigableMap)} instead
|
||||
*/
|
||||
@Deprecated
|
||||
@Override
|
||||
public Put setFamilyCellMap(NavigableMap<byte[], List<Cell>> map) {
|
||||
return (Put) super.setFamilyCellMap(map);
|
||||
}
|
||||
|
||||
@Override
|
||||
public Put setClusterIds(List<UUID> clusterIds) {
|
||||
return (Put) super.setClusterIds(clusterIds);
|
||||
|
|
|
@ -45,6 +45,7 @@ import org.apache.hadoop.hbase.client.Durability;
|
|||
import org.apache.hadoop.hbase.client.Mutation;
|
||||
import org.apache.hadoop.hbase.client.Put;
|
||||
import org.apache.hadoop.hbase.client.RegionInfo;
|
||||
import org.apache.hadoop.hbase.client.Row;
|
||||
import org.apache.hadoop.hbase.regionserver.HRegion;
|
||||
import org.apache.hadoop.hbase.regionserver.wal.AbstractFSWAL;
|
||||
import org.apache.hadoop.hbase.util.Bytes;
|
||||
|
@ -453,7 +454,7 @@ public final class WALSplitUtil {
|
|||
|
||||
@Override
|
||||
public int compareTo(final MutationReplay d) {
|
||||
return this.mutation.compareTo(d.mutation);
|
||||
return Row.COMPARATOR.compare(mutation, d.mutation);
|
||||
}
|
||||
|
||||
@Override
|
||||
|
|
|
@ -157,7 +157,7 @@ public class TestSerialReplicationChecker {
|
|||
}
|
||||
for (int i = 0; i < barriers.length; i++) {
|
||||
put.addColumn(HConstants.REPLICATION_BARRIER_FAMILY, HConstants.SEQNUM_QUALIFIER,
|
||||
put.getTimeStamp() - barriers.length + i, Bytes.toBytes(barriers[i]));
|
||||
put.getTimestamp() - barriers.length + i, Bytes.toBytes(barriers[i]));
|
||||
}
|
||||
try (Table table = UTIL.getConnection().getTable(TableName.META_TABLE_NAME)) {
|
||||
table.put(put);
|
||||
|
|
|
@ -189,7 +189,7 @@ public class TestHBaseFsckCleanReplicationBarriers {
|
|||
}
|
||||
for (int i = 0; i < barriers.length; i++) {
|
||||
put.addColumn(HConstants.REPLICATION_BARRIER_FAMILY, HConstants.SEQNUM_QUALIFIER,
|
||||
put.getTimeStamp() - barriers.length + i, Bytes.toBytes(barriers[i]));
|
||||
put.getTimestamp() - barriers.length + i, Bytes.toBytes(barriers[i]));
|
||||
}
|
||||
try (Table table = UTIL.getConnection().getTable(TableName.META_TABLE_NAME)) {
|
||||
table.put(put);
|
||||
|
|
Loading…
Reference in New Issue