Comments on streaming enhancement

Author: merbla

src/Serilog.Sinks.Splunk/Sinks/Splunk/EventCollectorRequest.cs

@@ -6,7 +6,16 @@ namespace Serilog.Sinks.Splunk
 {
     internal class EventCollectorRequest : HttpRequestMessage
     {
-        public EventCollectorRequest(
+        internal EventCollectorRequest(string splunkHost, string jsonPayLoad) 
+        {
+
+            var stringContent = new StringContent(jsonPayLoad, Encoding.UTF8, "application/json");
+            RequestUri = new Uri(splunkHost);
+            Content = stringContent;
+            Method = HttpMethod.Post;
+        }
+
+        internal EventCollectorRequest(
             string splunkHost, 
             string logEvent, 
             string source,

src/Serilog.Sinks.Splunk/Sinks/Splunk/EventCollectorSink.cs

@@ -153,8 +153,11 @@ private async Task ProcessQueue()
                     if (events.Count == 0)
                         return;
 
-                    //TODO: Add streaming capability for performance.  This should b
-                    
+                    //TODO: Add streaming capability for performance.  
+                    // - Stream writer needs to move to outer scope.
+                    // - Change Event Collector Request to only take string (or stream) of events and host
+                    // - New object to stream many events as collection
+
                     foreach (var logEvent in events)
                     {
                         var sw = new StringWriter();