jenkins-bot has submitted this change and it was merged.
Change subject: Support a single id with --ids
......................................................................
Support a single id with --ids
Change-Id: Idd044b77fd12dd38f9b76db6f359115157609855
---
M tools/src/main/java/org/wikidata/query/rdf/tool/Update.java
1 file changed, 30 insertions(+), 19 deletions(-)
Approvals:
Smalyshev: Looks good to me, approved
jenkins-bot: Verified
diff --git a/tools/src/main/java/org/wikidata/query/rdf/tool/Update.java
b/tools/src/main/java/org/wikidata/query/rdf/tool/Update.java
index 3882e59..b3f303d 100644
--- a/tools/src/main/java/org/wikidata/query/rdf/tool/Update.java
+++ b/tools/src/main/java/org/wikidata/query/rdf/tool/Update.java
@@ -56,7 +56,8 @@
@Option(shortName = "s", defaultToNull = true, description = "Start
time in 2015-02-11T17:11:08Z or 20150211170100 format.")
String start();
- @Option(defaultToNull = true, description = "If specified must be
<start>-<end>. Ids are iterated instead of recent changes. Start and end are
inclusive.")
+ @Option(defaultToNull = true, description = "If specified must be <id>
or <start>-<end>. Ids are iterated instead of recent "
+ + "changes. Start and end are inclusive.")
String ids();
@Option(shortName = "u", description = "URL to post updates and
queries.")
@@ -95,9 +96,8 @@
new LinkedBlockingQueue<Runnable>(), threadFactory.build());
Munger munger = CliUtils.mungerFromOptions(options);
- new Update<>(changeSource, wikibaseRepository, rdfRepository, munger,
executor).
- setPollDelay(options.pollDelay()).
- run();
+ new Update<>(changeSource, wikibaseRepository, rdfRepository, munger,
executor).setPollDelay(
+ options.pollDelay()).run();
}
/**
@@ -110,11 +110,22 @@
WikibaseRepository wikibaseRepository) {
if (options.ids() != null) {
String[] ids = options.ids().split("-");
- if (ids.length != 2) {
+ long start;
+ long end;
+ switch (ids.length) {
+ case 1:
+ start = Long.parseLong(ids[0]);
+ end = start;
+ break;
+ case 2:
+ start = Long.parseLong(ids[0]);
+ end = Long.parseLong(ids[1]);
+ break;
+ default:
log.error("Invalid format for --ids. Need <start>-<stop>.");
return null;
}
- return IdChangeSource.forItems(Long.parseLong(ids[0]),
Long.parseLong(ids[1]), 30);
+ return IdChangeSource.forItems(start, end, options.batchSize());
}
long startTime;
if (options.start() != null) {
@@ -227,19 +238,19 @@
do {
batchAdvanced.mark(batch.advanced());
log.info("Polled up to {} at {} updates per second and {}
{} per second", batch.leftOffHuman(),
- meterReport(updateMeter),
meterReport(batchAdvanced), batch.advancedUnits());
- if (batch.last()) {
- return;
- }
- batch = changeSource.nextBatch(batch);
- if(batch.changes().isEmpty()) {
- log.debug("Sleeping for {} secs", pollDelay);
- Thread.sleep(pollDelay*1000);
- } else {
- log.debug("{} changes in batch", batch.changes().size());
- break;
- }
- } while(true);
+ meterReport(updateMeter),
meterReport(batchAdvanced), batch.advancedUnits());
+ if (batch.last()) {
+ return;
+ }
+ batch = changeSource.nextBatch(batch);
+ if (batch.changes().isEmpty()) {
+ log.debug("Sleeping for {} secs", pollDelay);
+ Thread.sleep(pollDelay * 1000);
+ } else {
+ log.debug("{} changes in batch",
batch.changes().size());
+ break;
+ }
+ } while (true);
} catch (RetryableException e) {
log.warn("Error occurred during poll loop. Retrying.", e);
} catch (InterruptedException e) {
--
To view, visit https://gerrit.wikimedia.org/r/205854
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Idd044b77fd12dd38f9b76db6f359115157609855
Gerrit-PatchSet: 1
Gerrit-Project: wikidata/query/rdf
Gerrit-Branch: master
Gerrit-Owner: Manybubbles <[email protected]>
Gerrit-Reviewer: Jdouglas <[email protected]>
Gerrit-Reviewer: Smalyshev <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
