Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HBASE-22481: Javadoc Warnings reference not found #254

Merged
merged 4 commits into from
Jun 8, 2019
Merged

HBASE-22481: Javadoc Warnings reference not found #254

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
3eff922
HBASE-22481
murtazahassan123 May 27, 2019
e802d8c
HBASE-22481 Javadoc Warnings: reference not found, added an entry in …
murtazahassan123 May 27, 2019
61e1137
HBASE-22481 Changed suppresions.xml file with check on package instea…
murtazahassan123 May 28, 2019
2638f92
HBASE-22481: Javadoc Warnings reference not found, suppress LineLengh…
murtazahassan123 Jun 7, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions hbase-checkstyle/src/main/resources/hbase/checkstyle.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@

-->
<module name="Checker">
<!-- Filter out Checkstyle warnings that have been suppressed with the @SuppressWarnings
annotation -->
<module name="SuppressWarningsFilter"/>

<module name="FileTabCharacter"/>
<module name="TreeWalker">

Expand Down Expand Up @@ -130,5 +134,8 @@
http://checkstyle.sourceforge.net/config_whitespace.html -->
<module name="MethodParamPad"/>
<module name="ParenPad"/>

<!-- Make the @SuppressWarnings annotations available to Checkstyle -->
<module name="SuppressWarningsHolder"/>
</module>
</module>
27 changes: 14 additions & 13 deletions ...mapreduce/src/main/java/org/apache/hadoop/hbase/mapred/MultiTableSnapshotInputFormat.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@

package org.apache.hadoop.hbase.mapred;

import edu.umd.cs.findbugs.annotations.SuppressWarnings;
import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.Path;
import org.apache.yetus.audience.InterfaceAudience;
Expand All @@ -38,15 +39,15 @@
import java.util.Map;

/**
* MultiTableSnapshotInputFormat generalizes {@link org.apache.hadoop.hbase.mapred
* .TableSnapshotInputFormat}
* MultiTableSnapshotInputFormat generalizes
* {@link org.apache.hadoop.hbase.mapred.TableSnapshotInputFormat}
* allowing a MapReduce job to run over one or more table snapshots, with one or more scans
* configured for each.
* Internally, the input format delegates to {@link org.apache.hadoop.hbase.mapreduce
* .TableSnapshotInputFormat}
* and thus has the same performance advantages; see {@link org.apache.hadoop.hbase.mapreduce
* .TableSnapshotInputFormat} for
* more details.
* Internally, the input format delegates to
* {@link org.apache.hadoop.hbase.mapreduce.TableSnapshotInputFormat}
* and thus has the same performance advantages; see
* {@link org.apache.hadoop.hbase.mapreduce.TableSnapshotInputFormat}
* for more details.
* Usage is similar to TableSnapshotInputFormat, with the following exception:
* initMultiTableSnapshotMapperJob takes in a map
* from snapshot name to a collection of scans. For each snapshot in the map, each corresponding
Expand All @@ -71,8 +72,8 @@
* </pre>
* Internally, this input format restores each snapshot into a subdirectory of the given tmp
* directory. Input splits and
* record readers are created as described in {@link org.apache.hadoop.hbase.mapreduce
* .TableSnapshotInputFormat}
* record readers are created as described in
* {@link org.apache.hadoop.hbase.mapreduce.TableSnapshotInputFormat}
* (one per region).
* See {@link org.apache.hadoop.hbase.mapreduce.TableSnapshotInputFormat} for more notes on
* permissioning; the
Expand Down Expand Up @@ -107,13 +108,13 @@ public RecordReader<ImmutableBytesWritable, Result> getRecordReader(InputSplit s
return new TableSnapshotRecordReader((TableSnapshotRegionSplit) split, job);
}

@SuppressWarnings("checkstyle:linelength")
/**
* Configure conf to read from snapshotScans, with snapshots restored to a subdirectory of
* restoreDir.
* Sets: {@link org.apache.hadoop.hbase.mapreduce
* .MultiTableSnapshotInputFormatImpl#RESTORE_DIRS_KEY},
* {@link org.apache.hadoop.hbase.mapreduce
* .MultiTableSnapshotInputFormatImpl#SNAPSHOT_TO_SCANS_KEY}
* Sets:
* {@link org.apache.hadoop.hbase.mapreduce.MultiTableSnapshotInputFormatImpl#RESTORE_DIRS_KEY},
* {@link org.apache.hadoop.hbase.mapreduce.MultiTableSnapshotInputFormatImpl#SNAPSHOT_TO_SCANS_KEY}
*
* @param conf
* @param snapshotScans
Expand Down