Fix test_time_millis on Windows (#275)

* ensure converted to int64 before multiplication
use numpy array operator instead of loop

* ...and enable the test

* revert changes

* remove comment

fastparquet/converted_types.py

@@ -18,6 +18,8 @@
 import numpy as np
 import binascii
 
+import sys
+
 from .thrift_structures import parquet_thrift
 from .util import PY2
 from .speedups import array_decode_utf8
@@ -120,6 +122,8 @@ def from_bytes(d):
         return (data * DAYS_TO_MILLIS).view('datetime64[ns]')
     elif ctype == parquet_thrift.ConvertedType.TIME_MILLIS:
         out = np.empty(len(data), dtype='int64')
+        if sys.platform == 'win32':
+            data = data.astype('int64')
         time_shift(data, out, 1000000)
         return out.view('timedelta64[ns]')
     elif ctype == parquet_thrift.ConvertedType.TIMESTAMP_MILLIS:

fastparquet/test/test_converted_types.py

@@ -42,7 +42,6 @@ def test_date():
             pd.to_datetime([datetime.date(2004, 11, 3)]))
 
 
-@pytest.mark.skipif(sys.platform == 'win32' and PY2, reason='does not work on windows 32 py2.7')
 def test_time_millis():
     """Test int32 encoding a timedelta in millis."""
     schema = pt.SchemaElement(