Expose DataStreamWriter.Foreach API

src/csharp/Microsoft.Spark.E2ETest/IpcTests/Sql/Streaming/DataStreamWriterTests.cs

@@ -2,7 +2,11 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 // See the LICENSE file in the project root for more information.
 
+using System;
 using System.Collections.Generic;
+using System.IO;
+using System.Linq;
+using Microsoft.Spark.E2ETest.Utils;
 using Microsoft.Spark.Sql;
 using Microsoft.Spark.Sql.Streaming;
 using Xunit;
@@ -59,5 +63,204 @@ public void TestSignaturesV2_3_X()
 
             Assert.IsType<DataStreamWriter>(dsw.Trigger(Trigger.Once()));
         }
+
+        [SkipIfSparkVersionIsLessThan(Versions.V2_4_0)]
+        public void TestForeach()
+        {
+            // Temporary folder to put our test stream input.
+            using var srcTempDirectory = new TemporaryDirectory();
+            string streamInputPath = Path.Combine(srcTempDirectory.Path, "streamInput");
+
+            // [1, 2, ..., 99]
+            _spark.Range(1, 100).Write().Json(streamInputPath);
+
+            // Test a scenario where ForeachWriter runs without issues.
+            // If everything is working as expected, then:
+            // - Triggering stream will not throw an exception
+            // - 3 CSV files will be created in the temporary directory.
+            // - 0 Exception files will be created in the temporary directory.
+            // - The CSV files will contain valid data to read, where the
+            //   expected entries will contain [101, 102, ..., 199]
+            TestAndValidateForeach(
+                streamInputPath,
+                new TestForeachWriter(),
+                false,
+                3,
+                3,
+                0,
+                Enumerable.Range(101, 99));
+
+            // Test scenario where IForeachWriter.Open returns false.
+            // When IForeachWriter.Open returns false, then IForeachWriter.Process
+            // is not called.  Verify that:
+            // - Triggering stream will not throw an exception
+            // - 3 CSV files will be created in the temporary directory.
+            // - 0 Exception files will be created in the temporary directory.
+            // - The CSV files will not contain valid data to read.
+            TestAndValidateForeach(
+                streamInputPath,
+                new TestForeachWriterOpenFailure(),
+                false,
+                3,
+                3,
+                0,
+                Enumerable.Empty<int>());
+
+
+            // Test scenario where ForeachWriter.Process throws an Exception.
+            // When IForeachWriter.Process throws an Exception, then the exception
+            // is rethrown by ForeachWriterWrapper. Verify that:
+            // - Triggering stream throws an exception.
+            // - 1 CSV file will be created in the temporary directory.
+            // - 1 Exception will be created in the temporary directory. The
+            //   thrown exception from Process() will be sent to Close().
+            // - The CSV file will not contain valid data to read.
+            TestAndValidateForeach(
+                    streamInputPath,
+                    new TestForeachWriterProcessFailure(),
+                    true,
+                    1,
+                    1,
+                    1,
+                    Enumerable.Empty<int>());
+        }
+
+        private void TestAndValidateForeach(
+            string streamInputPath,
+            TestForeachWriter foreachWriter,
+            bool foreachThrows,
+            int partitions,
+            long expectedCSVFiles,
+            long expectedExceptionFiles,
+            IEnumerable<int> expectedOutput)
+        {
+            // Temporary folder the TestForeachWriter will write to.
+            using var dstTempDirectory = new TemporaryDirectory();
+
+            foreachWriter.WritePath = dstTempDirectory.Path;
+
+            // Read streamInputPath, repartitions data into `partitions`, then
+            // calls TestForeachWriter on the data.
+            DataStreamWriter dsw = _spark
+                .ReadStream()
+                .Schema("id INT")
+                .Json(streamInputPath)
+                .Repartition(partitions)
+                .WriteStream()
+                .Foreach(foreachWriter);
+
+            // Trigger the stream batch once.
+            if (foreachThrows)
+            {
+                Assert.Throws<Exception>(
+                    () => dsw.Trigger(Trigger.Once()).Start().AwaitTermination());
+            }
+            else
+            {
+                dsw.Trigger(Trigger.Once()).Start().AwaitTermination();
+            }
+
+            // Verify that TestForeachWriter created a unique .csv for each
+            // partition.
+            Assert.Equal(
+                expectedCSVFiles,
+                Directory.GetFiles(dstTempDirectory.Path, "*.csv").Length);
+
+            // When ForeachWriter.Process(Row) throws an exception,
+            // ForeachWriter.Close(Exception) will create a file with the
+            // .exeception extension.
+            Assert.Equal(
+                expectedExceptionFiles,
+                Directory.GetFiles(dstTempDirectory.Path, "*.exception").Length);
+
+            // Read in the *.csv file(s) generated by the TestForeachWriter.
+            // If there are multiple input files, sorting by "id" will make
+            // validation simpler.
+            DataFrame foreachWriterOutputDF = _spark
+                .Read()
+                .Schema("id INT")
+                .Csv(dstTempDirectory.Path)
+                .Sort("id");
+
+            // TestForeachWriterProcessFailure.Process(Row) did not write anything to
+            // the .csv files.
+            Assert.Equal(
+                    expectedOutput.Select(i => new object[] { i }),
+                    foreachWriterOutputDF.Collect().Select(r => r.Values));
+        }
+
+        [Serializable]
+        private class TestForeachWriter : IForeachWriter
+        {
+            // Mark the StreamWriter as ThreadStatic. If there are multiple Tasks
+            // running this ForeachWriter, on the same Worker, and in parallel, then
+            // it may be possible that they may use the same StreamWriter object. This
+            // may cause an unintended side effect of a Task writing the output to a
+            // file meant for a different Task.
+            [ThreadStatic]
+            private static StreamWriter s_streamWriter;
+
+            private long _partitionId;
+
+            private long _epochId;
+
+            internal string WritePath { get; set; }
+
+            public void Close(Exception errorOrNull)
+            {
+                if (errorOrNull != null)
+                {
+                    FileStream fs = File.Create(
+                        Path.Combine(
+                            WritePath,
+                            $"Close-{_partitionId}-{_epochId}.exception"));
+                    fs.Dispose();
+                }
+
+                s_streamWriter?.Dispose();
+            }
+
+            public virtual bool Open(long partitionId, long epochId)
+            {
+                _partitionId = partitionId;
+                _epochId = epochId;
+                try
+                {
+                    s_streamWriter = new StreamWriter(
+                        Path.Combine(
+                            WritePath,
+                            $"sink-foreachWriter-{_partitionId}-{_epochId}.csv"));
+                    return true;
+                }
+                catch
+                {
+                    return false;
+                }
+            }
+
+            public virtual void Process(Row value)
+            {
+                s_streamWriter.WriteLine(string.Join(",", value.Values.Select(v => 100 + (int)v)));
+            }
+        }
+
+        [Serializable]
+        private class TestForeachWriterOpenFailure : TestForeachWriter
+        {
+            public override bool Open(long partitionId, long epochId)
+            {
+                base.Open(partitionId, epochId);
+                return false;
+            }
+        }
+
+        [Serializable]
+        private class TestForeachWriterProcessFailure : TestForeachWriter
+        {
+            public override void Process(Row value)
+            {
+                throw new Exception("TestForeachWriterProcessFailure Process(Row) failure.");
+            }
+        }
     }
 }

src/csharp/Microsoft.Spark.Worker/Payload.cs

@@ -3,81 +3,10 @@
 // See the LICENSE file in the project root for more information.
 
 using System.Collections.Generic;
-using System.Linq;
 using Microsoft.Spark.Utils;
 
 namespace Microsoft.Spark.Worker
 {
-    /// <summary>
-    /// TaskContext stores information related to a task.
-    /// </summary>
-    internal class TaskContext
-    {
-        internal int StageId { get; set; }
-
-        internal int PartitionId { get; set; }
-
-        internal int AttemptNumber { get; set; }
-
-        internal long AttemptId { get; set; }
-
-        internal bool IsBarrier { get; set; }
-
-        internal int Port { get; set; }
-
-        internal string Secret { get; set; }
-
-        internal IEnumerable<Resource> Resources { get; set; } = new List<Resource>();
-
-        internal Dictionary<string, string> LocalProperties { get; set; } =
-            new Dictionary<string, string>();
-
-        public override bool Equals(object obj)
-        {
-            if (!(obj is TaskContext other))
-            {
-                return false;
-            }
-
-            return (StageId == other.StageId) &&
-                (PartitionId == other.PartitionId) &&
-                (AttemptNumber == other.AttemptNumber) &&
-                (AttemptId == other.AttemptId) &&
-                Resources.SequenceEqual(other.Resources) &&
-                (LocalProperties.Count == other.LocalProperties.Count) &&
-                !LocalProperties.Except(other.LocalProperties).Any();
-        }
-
-        public override int GetHashCode()
-        {
-            return StageId;
-        }
-
-        internal class Resource
-        {
-            internal string Key { get; set; }
-            internal string Value { get; set; }
-            internal IEnumerable<string> Addresses { get; set; } = new List<string>();
-
-            public override bool Equals(object obj)
-            {
-                if (!(obj is Resource other))
-                {
-                    return false;
-                }
-
-                return (Key == other.Key) &&
-                    (Value == other.Value) &&
-                    Addresses.SequenceEqual(Addresses);
-            }
-
-            public override int GetHashCode()
-            {
-                return Key.GetHashCode();
-            }
-        }
-    }
-
     /// <summary>
     /// BroadcastVariables stores information on broadcast variables.
     /// </summary>

src/csharp/Microsoft.Spark/Attributes.cs

@@ -73,4 +73,12 @@ public DeprecatedAttribute(string version)
         {
         }
     }
+
+    /// <summary>
+    /// Custom attribute to denote that a class is a Udf Wrapper.
+    /// </summary>
+    [AttributeUsage(AttributeTargets.Class)]
+    internal sealed class UdfWrapperAttribute : Attribute
+    {
+    }
 }

src/csharp/Microsoft.Spark/RDD.cs

@@ -261,16 +261,14 @@ public RDD<T> Sample(bool withReplacement, double fraction, long? seed = null)
         public IEnumerable<T> Collect()
         {
             (int port, string secret) = CollectAndServe();
-            using (ISocketWrapper socket = SocketFactory.CreateSocket())
-            {
-                socket.Connect(IPAddress.Loopback, port, secret);
+            using ISocketWrapper socket = SocketFactory.CreateSocket();
+            socket.Connect(IPAddress.Loopback, port, secret);
 
-                var collector = new RDD.Collector();
-                System.IO.Stream stream = socket.InputStream;
-                foreach (T element in collector.Collect(stream, _serializedMode).Cast<T>())
-                {
-                    yield return element;
-                }
+            var collector = new RDD.Collector();
+            System.IO.Stream stream = socket.InputStream;
+            foreach (T element in collector.Collect(stream, _serializedMode).Cast<T>())
+            {
+                yield return element;
             }
         }
 
@@ -341,6 +339,7 @@ private JvmObjectReference GetJvmRef()
         /// </summary>
         /// <typeparam name="TArg">Input type</typeparam>
         /// <typeparam name="TResult">Output type</typeparam>
+        [UdfWrapper]
         internal sealed class MapUdfWrapper<TArg, TResult>
         {
             private readonly Func<TArg, TResult> _func;
@@ -361,6 +360,7 @@ internal IEnumerable<object> Execute(int pid, IEnumerable<object> input)
         /// </summary>
         /// <typeparam name="TArg">Input type</typeparam>
         /// <typeparam name="TResult">Output type</typeparam>
+        [UdfWrapper]
         internal sealed class FlatMapUdfWrapper<TArg, TResult>
         {
             private readonly Func<TArg, IEnumerable<TResult>> _func;
@@ -382,6 +382,7 @@ internal IEnumerable<object> Execute(int pid, IEnumerable<object> input)
         /// </summary>
         /// <typeparam name="TArg">Input type</typeparam>
         /// <typeparam name="TResult">Output type</typeparam>
+        [UdfWrapper]
         internal sealed class MapPartitionsUdfWrapper<TArg, TResult>
         {
             private readonly Func<IEnumerable<TArg>, IEnumerable<TResult>> _func;
@@ -403,6 +404,7 @@ internal IEnumerable<object> Execute(int pid, IEnumerable<object> input)
         /// </summary>
         /// <typeparam name="TArg">Input type</typeparam>
         /// <typeparam name="TResult">Output type</typeparam>
+        [UdfWrapper]
         internal sealed class MapPartitionsWithIndexUdfWrapper<TArg, TResult>
         {
             private readonly Func<int, IEnumerable<TArg>, IEnumerable<TResult>> _func;
@@ -423,9 +425,11 @@ internal IEnumerable<object> Execute(int pid, IEnumerable<object> input)
         /// Helper to map the UDF for Filter() to
         /// <see cref="RDD.WorkerFunction.ExecuteDelegate"/>.
         /// </summary>
+        [UdfWrapper]
         internal class FilterUdfWrapper
         {
             private readonly Func<T, bool> _func;
+
             internal FilterUdfWrapper(Func<T, bool> func)
             {
                 _func = func;