You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by xh...@apache.org on 2019/02/20 22:38:06 UTC
[incubator-pinot] branch master updated: Minor fix for integration
test (#3858)
This is an automated email from the ASF dual-hosted git repository.
xhsun pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pinot.git
The following commit(s) were added to refs/heads/master by this push:
new c943843 Minor fix for integration test (#3858)
c943843 is described below
commit c94384389923302a6e46fbedcc5825d766b6edad
Author: Xiaohui Sun <xh...@linkedin.com>
AuthorDate: Wed Feb 20 14:38:01 2019 -0800
Minor fix for integration test (#3858)
* [TE] detection - minor fixes for integration test
* [TE] backend - use dataset time zone as default for detector
* fix the timezone logic to use dataset timezone by default
---
.../apache/pinot/thirdeye/detection/wrapper/AnomalyDetectorWrapper.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/wrapper/AnomalyDetectorWrapper.java b/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/wrapper/AnomalyDetectorWrapper.java
index 5e7227f..0f1aab7 100644
--- a/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/wrapper/AnomalyDetectorWrapper.java
+++ b/thirdeye/thirdeye-pinot/src/main/java/org/apache/pinot/thirdeye/detection/wrapper/AnomalyDetectorWrapper.java
@@ -126,7 +126,7 @@ public class AnomalyDetectorWrapper extends DetectionPipeline {
this.dataset = this.provider.fetchDatasets(Collections.singletonList(metricConfigDTO.getDataset()))
.get(metricConfigDTO.getDataset());
// date time zone for moving windows. use dataset time zone as default
- this.dateTimeZone = DateTimeZone.forID(MapUtils.getString(config.getProperties(), PROP_TIMEZONE, "America/Los_Angeles"));
+ this.dateTimeZone = DateTimeZone.forID(MapUtils.getString(config.getProperties(), PROP_TIMEZONE, this.dataset.getTimezone()));
String bucketStr = MapUtils.getString(config.getProperties(), PROP_BUCKET_PERIOD);
this.bucketPeriod = bucketStr == null ? this.getBucketSizePeriodForDataset() : Period.parse(bucketStr);
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org