HBASE-13109 Fix Javadoc warning; and some misc checkstyle warnings
This commit is contained in:
parent
07cb6d2413
commit
1cdcb6e9b8
|
@ -76,8 +76,6 @@ public class ExplicitColumnTracker implements ColumnTracker {
|
||||||
* @param maxVersions maximum versions to return per column
|
* @param maxVersions maximum versions to return per column
|
||||||
* @param oldestUnexpiredTS the oldest timestamp we are interested in,
|
* @param oldestUnexpiredTS the oldest timestamp we are interested in,
|
||||||
* based on TTL
|
* based on TTL
|
||||||
* @param lookAhead number of KeyValues to look ahead via next before
|
|
||||||
* (re)seeking
|
|
||||||
*/
|
*/
|
||||||
public ExplicitColumnTracker(NavigableSet<byte[]> columns, int minVersions,
|
public ExplicitColumnTracker(NavigableSet<byte[]> columns, int minVersions,
|
||||||
int maxVersions, long oldestUnexpiredTS) {
|
int maxVersions, long oldestUnexpiredTS) {
|
||||||
|
|
|
@ -233,8 +233,9 @@ public class ScanQueryMatcher {
|
||||||
* @throws IOException
|
* @throws IOException
|
||||||
*/
|
*/
|
||||||
public ScanQueryMatcher(Scan scan, ScanInfo scanInfo, NavigableSet<byte[]> columns,
|
public ScanQueryMatcher(Scan scan, ScanInfo scanInfo, NavigableSet<byte[]> columns,
|
||||||
long readPointToUse, long earliestPutTs, long oldestUnexpiredTS, long now, byte[] dropDeletesFromRow,
|
long readPointToUse, long earliestPutTs, long oldestUnexpiredTS, long now,
|
||||||
byte[] dropDeletesToRow, RegionCoprocessorHost regionCoprocessorHost) throws IOException {
|
byte[] dropDeletesFromRow, byte[] dropDeletesToRow,
|
||||||
|
RegionCoprocessorHost regionCoprocessorHost) throws IOException {
|
||||||
this(scan, scanInfo, columns, ScanType.COMPACT_RETAIN_DELETES, readPointToUse, earliestPutTs,
|
this(scan, scanInfo, columns, ScanType.COMPACT_RETAIN_DELETES, readPointToUse, earliestPutTs,
|
||||||
oldestUnexpiredTS, now, regionCoprocessorHost);
|
oldestUnexpiredTS, now, regionCoprocessorHost);
|
||||||
Preconditions.checkArgument((dropDeletesFromRow != null) && (dropDeletesToRow != null));
|
Preconditions.checkArgument((dropDeletesFromRow != null) && (dropDeletesToRow != null));
|
||||||
|
@ -579,8 +580,6 @@ public class ScanQueryMatcher {
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @param nextIndexed the key of the next entry in the block index (if any)
|
* @param nextIndexed the key of the next entry in the block index (if any)
|
||||||
* @param off
|
|
||||||
* @param len
|
|
||||||
* @param kv The Cell we're using to calculate the seek key
|
* @param kv The Cell we're using to calculate the seek key
|
||||||
* @return result of the compare between the indexed key and the key portion of the passed cell
|
* @return result of the compare between the indexed key and the key portion of the passed cell
|
||||||
*/
|
*/
|
||||||
|
@ -594,8 +593,6 @@ public class ScanQueryMatcher {
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @param nextIndexed the key of the next entry in the block index (if any)
|
* @param nextIndexed the key of the next entry in the block index (if any)
|
||||||
* @param off
|
|
||||||
* @param len
|
|
||||||
* @param kv The Cell we're using to calculate the seek key
|
* @param kv The Cell we're using to calculate the seek key
|
||||||
* @return result of the compare between the indexed key and the key portion of the passed cell
|
* @return result of the compare between the indexed key and the key portion of the passed cell
|
||||||
*/
|
*/
|
||||||
|
|
|
@ -338,7 +338,7 @@ public class StoreScanner extends NonReversedNonLazyKeyValueScanner
|
||||||
}
|
}
|
||||||
scanner.seek(seekKey);
|
scanner.seek(seekKey);
|
||||||
Cell c = scanner.peek();
|
Cell c = scanner.peek();
|
||||||
if (c != null ) {
|
if (c != null) {
|
||||||
totalScannersSoughtBytes += CellUtil.estimatedSerializedSizeOf(c);
|
totalScannersSoughtBytes += CellUtil.estimatedSerializedSizeOf(c);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -604,7 +604,8 @@ public class StoreScanner extends NonReversedNonLazyKeyValueScanner
|
||||||
*/
|
*/
|
||||||
private ScanQueryMatcher.MatchCode optimize(ScanQueryMatcher.MatchCode qcode, Cell cell) {
|
private ScanQueryMatcher.MatchCode optimize(ScanQueryMatcher.MatchCode qcode, Cell cell) {
|
||||||
Cell nextIndexedKey = getNextIndexedKey();
|
Cell nextIndexedKey = getNextIndexedKey();
|
||||||
if (nextIndexedKey == null || nextIndexedKey == HConstants.NO_NEXT_INDEXED_KEY || store == null) {
|
if (nextIndexedKey == null || nextIndexedKey == HConstants.NO_NEXT_INDEXED_KEY ||
|
||||||
|
store == null) {
|
||||||
return qcode;
|
return qcode;
|
||||||
}
|
}
|
||||||
switch(qcode) {
|
switch(qcode) {
|
||||||
|
|
Loading…
Reference in New Issue