Skip to content

Commit f20b403

Browse files
DmitryLitvintsevmksahakyan
authored andcommitted
bulk: remove residual code having to do with delay_clear
Motivation: ---------- The delay option has been deprecated (e83f39a, https://rb.dcache.org/r/13767/) But apparently some code using this option has not been completely excised. There is issue #7858 opened Modification: ------------ Remove all code related to this feature Results: ------- No stack trace Ticket: https://github.com/dCache/dcache/issues/7858A Patch: https://rb.dcache.org/r/14509/ Acked-by: Tigran Target: trunk Request: 11.x Request: 10.x Request: 9.2 Require-book: no Require-notes: yes
1 parent ef6774c commit f20b403

File tree

5 files changed

+5
-29
lines changed

5 files changed

+5
-29
lines changed

modules/dcache-bulk/src/main/java/org/dcache/services/bulk/BulkServiceCommands.java

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -444,10 +444,6 @@ abstract class FilteredRequest implements Callable<Serializable> {
444444
usage = "Whether the request has this set to true or false.")
445445
Boolean clearOnFailure;
446446

447-
@Option(name = "delayClear",
448-
usage = "True means the request has a non-zero value of this.")
449-
Boolean delayClear;
450-
451447
@Option(name = "expandDirectories",
452448
valueSpec = "NONE|TARGETS|ALL",
453449
usage = "The recursion depth of the request.")
@@ -489,8 +485,8 @@ protected void configureFilters() throws ParseException {
489485
}
490486

491487
rFilter = new BulkRequestFilter(beforeStart, afterStart, owners, urlPrefixes, ids,
492-
activities, statuses, cancelOnFailure, clearOnSuccess, clearOnFailure, delayClear,
493-
depth);
488+
activities, statuses, cancelOnFailure, clearOnSuccess, clearOnFailure,
489+
depth);
494490
rFilter.setId(id);
495491
}
496492

@@ -524,7 +520,6 @@ private boolean emptyFilter() {
524520
&& activity == null
525521
&& clearOnFailure == null
526522
&& clearOnSuccess == null
527-
&& delayClear == null
528523
&& expandDirectories == null
529524
&& status == null;
530525
}

modules/dcache-bulk/src/main/java/org/dcache/services/bulk/store/jdbc/request/JdbcBulkRequestCriterion.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -200,13 +200,6 @@ public JdbcBulkRequestCriterion clearOnSuccess(Boolean clearOnSuccess) {
200200
return this;
201201
}
202202

203-
public JdbcBulkRequestCriterion delayClear(Boolean delayClear) {
204-
if (delayClear != null) {
205-
addClause("delay_clear = ?", delayClear);
206-
}
207-
return this;
208-
}
209-
210203
public JdbcBulkRequestCriterion expandDirectories(Depth depth) {
211204
if (depth != null) {
212205
addClause("expand_directories = ?", depth.name());
@@ -249,7 +242,6 @@ public JdbcBulkRequestCriterion filter(BulkRequestFilter filter) {
249242
cancelOnFailure(filter.getCancelOnFailure());
250243
clearOnFailure(filter.getClearOnFailure());
251244
clearOnSuccess(filter.getClearOnSuccess());
252-
delayClear(filter.getDelayClear());
253245
expandDirectories(filter.getExpandDirectories());
254246
activity(filter.getActivity());
255247
status(filter.getStatuses());

modules/dcache-bulk/src/main/java/org/dcache/services/bulk/store/jdbc/request/JdbcBulkRequestStore.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -269,7 +269,7 @@ public void clearWhenTerminated(Subject subject, String uid)
269269
String key = checkRequestPermissions(subject, uid);
270270
LOGGER.trace("clearWhenTerminated {}, {}.", key, uid);
271271
requestDao.update(requestDao.where().uids(uid),
272-
requestDao.set().clearOnFailure(true).clearOnSuccess(true).delayClear(0));
272+
requestDao.set().clearOnFailure(true).clearOnSuccess(true));
273273
requestCache.invalidate(uid);
274274
}
275275

modules/dcache-bulk/src/main/java/org/dcache/services/bulk/store/jdbc/request/JdbcBulkRequestUpdate.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -172,11 +172,6 @@ public JdbcBulkRequestUpdate cancelOnFailure(boolean cancelOnFailure) {
172172
return this;
173173
}
174174

175-
public JdbcBulkRequestUpdate delayClear(int delayClear) {
176-
set("delay_clear", delayClear);
177-
return this;
178-
}
179-
180175
public JdbcBulkRequestUpdate depth(Depth depth) {
181176
if (depth != null) {
182177
set("expand_directories", depth.name());

modules/dcache-bulk/src/main/java/org/dcache/services/bulk/util/BulkRequestFilter.java

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -78,29 +78,27 @@ public final class BulkRequestFilter {
7878
private final Boolean cancelOnFailure;
7979
private final Boolean clearOnSuccess;
8080
private final Boolean clearOnFailure;
81-
private final Boolean delayClear;
8281
private final Depth expandDirectories;
8382

8483
private Long id;
8584

8685
public BulkRequestFilter(Set<BulkRequestStatus> statuses) {
8786
this(null, null, null, null, null, null,
8887
statuses, null, null,
89-
null, null, null);
88+
null, null);
9089
}
9190

9291
public BulkRequestFilter(Long before, Long after, Set<String> owner, Set<String> urlPrefix,
9392
Set<String> uuid, Set<String> activity, Set<BulkRequestStatus> statuses,
9493
Boolean cancelOnFailure, Boolean clearOnSuccess, Boolean clearOnFailure,
95-
Boolean delayClear, Depth expandDirectories) {
94+
Depth expandDirectories) {
9695
this.before = before;
9796
this.after = after;
9897
this.activity = activity;
9998
this.statuses = statuses;
10099
this.cancelOnFailure = cancelOnFailure;
101100
this.clearOnSuccess = clearOnSuccess;
102101
this.clearOnFailure = clearOnFailure;
103-
this.delayClear = delayClear;
104102
this.expandDirectories = expandDirectories;
105103
this.owner = owner;
106104
this.urlPrefix = urlPrefix;
@@ -131,10 +129,6 @@ public Boolean getClearOnFailure() {
131129
return clearOnFailure;
132130
}
133131

134-
public Boolean getDelayClear() {
135-
return delayClear;
136-
}
137-
138132
public Depth getExpandDirectories() {
139133
return expandDirectories;
140134
}

0 commit comments

Comments
 (0)