sebastian-nagel commented on code in PR #974:
URL: https://github.com/apache/nutch/pull/974#discussion_r4238209138
##########
src/java/org/apache/nutch/publisher/NutchPublishers.java:
##########
@@ -32,7 +32,7 @@ public class NutchPublishers extends Configured implements
NutchPublisher{
private Configuration conf;
public NutchPublishers(Configuration conf) {
- this.conf = conf;
+ this.conf = conf;
Review Comment:
Same as above: indentation lost.
##########
src/java/org/apache/nutch/crawl/CrawlDb.java:
##########
@@ -342,12 +342,12 @@ else if(args.containsKey(Nutch.ARG_SEGMENTS)) {
Object segments = args.get(Nutch.ARG_SEGMENTS);
ArrayList<String> segmentList = new ArrayList<>();
if(segments instanceof ArrayList) {
- segmentList = (ArrayList<String>)segments;
+ segmentList = (ArrayList<String>)segments;
Review Comment:
Looks like here the code formatting should be applied. The indentation is
now lost.
But also the preceding line `if(segments ...` does follow the code
formatting rules.
##########
src/java/org/apache/nutch/util/ProtocolStatusStatistics.java:
##########
@@ -50,10 +50,10 @@
* An example output run showing the number of encountered status
* codes such as 200, 300, and a count of un-fetched record.
*
- * 38 200
Review Comment:
Should be inside `<pre>`
--
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]
