Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(568)

Issue 319730043: [plaso] JSON serializer support for AttributeContainers inside AttributeContainers #1085

Can't Edit
Can't Publish+Mail
Start Review
Created:
7 years, 5 months ago by dc3.plaso
Modified:
6 years, 3 months ago
CC:
kiddi, log2timeline-dev_googlegroups.com
Visibility:
Public.

Description

[plaso] JSON serializer support for AttributeContainers inside AttributeContainers #1085

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -9 lines) Patch
M plaso/serializer/json_serializer.py View 4 chunks +24 lines, -7 lines 0 comments Download
M tests/serializer/json_serializer.py View 5 chunks +28 lines, -2 lines 0 comments Download

Messages

Total messages: 8
dc3.plaso
7 years, 5 months ago (2016-11-14 18:00:46 UTC) #1
Joachim Metz
thanks for the changes, however I'll need to evaluate them in light of changes of ...
7 years, 5 months ago (2016-11-24 16:09:40 UTC) #2
dc3.plaso
On 2016/11/24 16:09:40, Joachim Metz wrote: > thanks for the changes, however I'll need to ...
7 years, 4 months ago (2016-12-06 13:59:49 UTC) #3
dc3.plaso
On 2016/11/24 16:09:40, Joachim Metz wrote: > thanks for the changes, however I'll need to ...
6 years, 11 months ago (2017-05-09 13:29:52 UTC) #4
Joachim Metz
I'll try to have a look shortly.
6 years, 11 months ago (2017-05-09 13:41:53 UTC) #5
Joachim Metz
Let me give this some thought. One of the idea I'm playing with is not ...
6 years, 11 months ago (2017-05-10 05:48:40 UTC) #6
dc3.plaso
On 2017/05/10 05:48:40, Joachim Metz wrote: > Let me give this some thought. One of ...
6 years, 11 months ago (2017-05-10 16:34:46 UTC) #7
Joachim Metz
6 years, 3 months ago (2018-01-26 06:34:43 UTC) #8
Per recent changes to SQLite storage. I'll not add the functionality to store
attribute containers (AC) inside ACs but prefer to use AC identifier approach
instead. Such as with event data. This approach is still evolving but reduces
the needed storage and should allow us to use more relational database features.

Sorry for the (very) slow response on this but I'll not merge these changes, but
thanks for providing them to us.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b