LogRecord should not expose internal data structures to processors/exporters

[cijothomas]

LogRecord currently exposes attributes as a Vec , which would prevent us from changing the implementation later without breaking processors/exporters. We can expose iterators only for attributes, allowing us the flexibility to change implementation in the future freely.