Github user chemikadze commented on a diff in the pull request:

    https://github.com/apache/incubator-griffin/pull/435#discussion_r224981272
  
    --- Diff: ui/angular/src/app/job/create-job/batch/batch.component.ts ---
    @@ -134,8 +134,8 @@ export class BatchComponent implements OnInit, 
AfterViewChecked {
           return false;
         }
         this.measureid = this.getMeasureId();
    -    let time = new Date().getTimezoneOffset() / 60;
    -    let timezone = "GMT" + time + ":00";
    +    let time = -(new Date().getTimezoneOffset() / 60);
    --- End diff --
    
    > IC, this timezone is used for data partition.
    
    As far as I see from code and behavior, this time zone is used only to 
interpret cron expression.
    Data partitions are referring to timezones configured on datasets 
themselves.


---

Reply via email to