Fix string deserialize exception

[tuhuynh27]

Fixes #73

For String byte[], it doesn't follow the java serialization protocol, thus a SerializationException will be thrown when deserializing on String byte[]

[the123saurav]

Should we wrap the SerializationException thrown during deserilise to a custom one, so that we can distinguish it from the same thrown in call to serialise?

[tuhuynh27]

Hmm I didn't get your idea, please suggest changes

[tuhuynh27]

Like we need to wrap SerializationException and ClassCastException to something like KevaTypeException right?

[the123saurav]

I meant we have code like this:

try {
            byte[] value = chronicleMap.get(key);
            LinkedList<BytesValue> list;
            list = value == null ? new LinkedList<>() : (LinkedList<BytesValue>) SerializationUtils.deserialize(value); <- can throw
            for (byte[] v : values) {
                list.addFirst(new BytesValue(v));
            }
            chronicleMap.put(key, SerializationUtils.serialize(list)); <- can throw
            return list.size();
        } finally {
            lock.unlock();
        }

From 2 places same exception can be thrown but only 1 of them indicates client error.
Although I do agree that serislisation should never throw, but still!

[tuhuynh27]

Sure @the123saurav, this is a good suggestion. But it will need to add extra code to every operation method, so I think we should create another issue for that refactoring, and put it to the backlog, after the first release we can resume on it

[the123saurav]

SOunds good!

[tuhuynh27]

Hi @the123saurav, created #75 backlog issue for your suggestion