Ottomata has submitted this change and it was merged.

Change subject: Show status of daily datasets, even if only part of day is 
dumped
......................................................................


Show status of daily datasets, even if only part of day is dumped

Change-Id: I538032494a40e0c27b5a40e5ed2c4e44d023acc0
---
M bin/refinery-dump-status-webrequest-partitions
1 file changed, 4 insertions(+), 3 deletions(-)

Approvals:
  Ottomata: Verified; Looks good to me, approved



diff --git a/bin/refinery-dump-status-webrequest-partitions 
b/bin/refinery-dump-status-webrequest-partitions
index 51348ce..a72937c 100755
--- a/bin/refinery-dump-status-webrequest-partitions
+++ b/bin/refinery-dump-status-webrequest-partitions
@@ -440,7 +440,8 @@
     # Check if this hour produces output at all. If not, omit it
     # completely. This omitting allows to not get superfluous 23 empty lines,
     # when only daily datasets are visible.
-    if [ \( "$HAS_VISIBLE_DAILY_DATASETS" = yes -a "${DATE: -2}" = "00" \) \
+    if [ \( "$HAS_VISIBLE_DAILY_DATASETS" = yes -a \
+            \( "${DATE: -2}" = "00" -o "$HOURS_OFFSET" = "$HOUR_OFFSET_MAX" \) 
\) \
         -o "$HAS_VISIBLE_HOURLY_DATASETS" = yes ]
     then
 
@@ -449,7 +450,7 @@
         # daily datasets first
         if [ "$HAS_VISIBLE_DAILY_DATASETS" = yes ]
         then
-            if [ "${DATE: -2}" = "00" ]
+            if [ "${DATE: -2}" = "00" -o "$HOURS_OFFSET" = "$HOUR_OFFSET_MAX" ]
             then
                 log_no_lf "|| ${DATE:0:10}/1D ||"
             else
@@ -460,7 +461,7 @@
             do
                 if [ "${DATASET_VISIBILITIES["$DATASET"]}" = "yes" -a 
"${DATASET_RECURRENCES["$DATASET"]}" = "daily" ]
                 then
-                    if [ "${DATE: -2}" = "00" ]
+                    if [ "${DATE: -2}" = "00"  -o "$HOURS_OFFSET" = 
"$HOUR_OFFSET_MAX" ]
                     then
                         dump_dataset_$DATASET "$DATE"
                     else

-- 
To view, visit https://gerrit.wikimedia.org/r/188006
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I538032494a40e0c27b5a40e5ed2c4e44d023acc0
Gerrit-PatchSet: 1
Gerrit-Project: analytics/refinery
Gerrit-Branch: master
Gerrit-Owner: QChris <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to