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 <[email protected]>
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: [email protected]
For additional commands, e-mail: [email protected]
