refactor: clean up parts of the codebase

[de-sh]

Fixes #XXXX.

Summary

• De-serialize Mode, don't compare strings.
• to_string runs to_vec, so just do that.
• Custom SSEError type for readability.
• Fewer hard-coded serde renames.
• fix: there are a 1000milis every sec.
• De-serialize Compression to not hard-code matching.
• Space used/available in hottier is not optional.
• Only construct the ObjectStorage client once.
• Reorganize as methods to save on cost of copy.
• Mutate only the file_path on windows.
• DiskUtil replaces the tuple (total_disk_space, available_disk_space, used_disk_space).
• Save on cost of cloning within EventFormat

---

This PR has:

• been tested to ensure log ingestion and log query works.
• added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
• added documentation for new or modified features or behaviors.

[github-actions]

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

[de-sh]

I have read the CLA Document and I hereby sign the CLA

[coveralls]

Pull Request Test Coverage Report for Build 12433752181

Details

• 50 of 500 (10.0%) changed or added relevant lines in 21 files are covered.
• 9 unchanged lines in 4 files lost coverage.
• Overall coverage increased (+0.06%) to 11.595%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/analytics.rs	0	1	0.0%
src/storage/azure_blob.rs	0	1	0.0%
src/storage/localfs.rs	0	1	0.0%
src/storage/staging.rs	0	1	0.0%
src/event/format/mod.rs	8	10	80.0%
src/handlers/http/modal/ingest_server.rs	0	2	0.0%
src/storage/object_storage.rs	0	3	0.0%
src/handlers/http/logstream.rs	0	4	0.0%
src/utils/json/mod.rs	13	17	76.47%
src/cli.rs	0	6	0.0%

Files with Coverage Reduction	New Missed Lines	%
src/storage/s3.rs	1	0.0%
src/handlers/http/logstream.rs	1	1.56%
src/query/stream_schema_provider.rs	2	26.9%
src/hottier.rs	5	0.0%

Totals
Change from base Build 12411844504:	0.06%
Covered Lines:	2092
Relevant Lines:	18043

---

💛 - Coveralls

[nitisht]

LGTM!