Github user paul-rogers commented on a diff in the pull request:

    https://github.com/apache/drill/pull/756#discussion_r103000211
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/profile/FragmentWrapper.java
 ---
    @@ -35,6 +38,10 @@
     public class FragmentWrapper {
       private final MajorFragmentProfile major;
       private final long start;
    +  private final Locale currentLocale = Locale.getDefault();
    +  private final String pattern = "dd-MMM-yyyy HH:mm:ss";
    +  private final SimpleDateFormat simpleDateFormat = new SimpleDateFormat(
    --- End diff --
    
    The following is the same:
    ```
    new SimpleDateFormat(pattern);
    ```
    
    Note that the locale info *does not* adjust the time zone. Local time is 
quite clunky with the original Java classes; we need to use the Joda classes to 
better handle local date/times.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to