[PATCH mogilefs 2/7] Clean up two instances of UNIX_TIMESTAMP()
that lurked outside the Store classes. $self->unix_timestamp
should used instead.
Robin H. Johnson
robbat2 at gentoo.org
Tue Jun 5 13:38:10 UTC 2007
From: Robin H. Johnson <robbat2 at gentoo.org>
Signed-off-by: Robin H. Johnson <robbat2 at gentoo.org>
---
server/lib/MogileFS/Worker/Delete.pm | 2 +-
server/lib/MogileFS/Worker/Query.pm | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/server/lib/MogileFS/Worker/Delete.pm b/server/lib/MogileFS/Worker/Delete.pm
index 3b439e3..4838c48 100644
--- a/server/lib/MogileFS/Worker/Delete.pm
+++ b/server/lib/MogileFS/Worker/Delete.pm
@@ -156,7 +156,7 @@ sub process_deletes {
my $reschedule_fid = sub {
my ($secs, $reason) = (int(shift), shift);
$dbh->do("INSERT IGNORE INTO file_to_delete_later (fid, delafter) ".
- "VALUES (?,UNIX_TIMESTAMP()+$secs)", undef,
+ "VALUES (?,".$self->unix_timestamp."+$secs)", undef,
$fid);
$sto->condthrow("Failure to insert into file_to_delete_later");
error("delete of fid $fid rescheduled: $reason") if $Mgd::DEBUG >= 2;
diff --git a/server/lib/MogileFS/Worker/Query.pm b/server/lib/MogileFS/Worker/Query.pm
index c5e9ab8..abeaada 100644
--- a/server/lib/MogileFS/Worker/Query.pm
+++ b/server/lib/MogileFS/Worker/Query.pm
@@ -1014,7 +1014,7 @@ sub cmd_stats {
$ret->{"replicationcount"} = $count;
# now we want to do the "new" replication stats
- my $db_time = $dbh->selectrow_array('SELECT UNIX_TIMESTAMP()');
+ my $db_time = $dbh->selectrow_array('SELECT '.$self->unix_timestamp);
$stats = $dbh->selectall_arrayref('SELECT nexttry, COUNT(*) FROM file_to_replicate GROUP BY 1');
foreach my $stat (@$stats) {
if ($stat->[0] < 1000) {
--
1.5.2
More information about the mogilefs
mailing list