SerializationError for datetime.time object

JSONSerializer does not support time type.

elasticsearch/serializer.py

class JSONSerializer(object):
    mimetype = 'application/json'

    def default(self, data):
        if isinstance(data, (date, datetime)):
            return data.isoformat()
        elif isinstance(data, Decimal):
            return float(data)
        raise TypeError("Unable to serialize %r (type: %s)" % (data, type(data)))

so I get a following error:

TypeError("Unable to serialize datetime.time
(10, 34) (type: <type 'datetime.time'>)"

A fix is quick and easy. Return time in isoformat like for date and datetime types:

class JSONSerializer(object):
    mimetype = 'application/json'

    def default(self, data):
        if isinstance(data, (date, time, datetime)):
            return data.isoformat()
        elif isinstance(data, Decimal):
            return float(data)
        raise TypeError("Unable to serialize %r (type: %s)" % (data, type(data)))

[honzakral]

@zeinababbasi there is no need to serialize a datetime object, elasticsearch-py will do it for you automatically (by using isoformat()) - see https://github.com/elastic/elasticsearch-py/blob/master/elasticsearch/serializer.py#L29 for details

[zeinababbasi]

@HonzaKral, thank you. I’ve tried it yesterday with isoformat(). BTW, I appreciate your response in a thread which is closed for 2 and a half years.