From TODO in src/main/java/io/archivesunleashed/data/WarcRecordUtils.java and src/main/java/io/archivesunleashed/data/ArcRecordUtils.java that will be removed with the resolution of #3:
These methods work fine, but there's a lot of unnecessary buffer copying, which is terrible from a performance perspective.
Do we see ourselves as tackling this?
I think we need to leave this open, if that is what you're asking.
@lintool should be the decider on closing this or leaving it open, since it looks like he added it https://github.com/lintool/warcbase/blame/cab311ed8b0bceb666865fa76dd3bc5a86402e13/warcbase-core/src/main/java/org/warcbase/data/WarcRecordUtils.java#L40
Yes, I think we should leave this open. Something we should tackle when (if...) we have cycles...
Most helpful comment
Yes, I think we should leave this open. Something we should tackle when (if...) we have cycles...