[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: remove 'graphs' and mark as 'future'?
The SARIF v2 graphs work is inconsistent with other changes that we’ve recently made. Specifically, it is one of the last remaining dictionaries in the format. To be consistent with the remaining design, we should consider converting this data to arrays, as we have elsewhere (we’ve done this for various performance reasons, primarily). At this late stage, though, cycles are few and no one, to my knowledge, has actually attempted to produce/consume this feature.
We have three options:
Any thoughts on this? My sense is that I personally insisted on getting this feature designed based on a concern that the format was insufficient for Semmle’s purposes. If you won’t actually utilize/find value in the feature, that undercuts the argument for further investment (and supports removing it for now).