From a4ba0303d881d3b733d02eba3ad4dfce6882043f Mon Sep 17 00:00:00 2001
From: Fengda Huang <h@phodal.com>
Date: Fri, 17 Feb 2023 14:52:36 +0800
Subject: [PATCH 1/3] Create 0004-apache-arrow-as-data-format.md

---
 text/0004-apache-arrow-as-data-format.md | 102 +++++++++++++++++++++++
 1 file changed, 102 insertions(+)
 create mode 100644 text/0004-apache-arrow-as-data-format.md

diff --git a/text/0004-apache-arrow-as-data-format.md b/text/0004-apache-arrow-as-data-format.md
new file mode 100644
index 0000000..f4d680e
--- /dev/null
+++ b/text/0004-apache-arrow-as-data-format.md
@@ -0,0 +1,102 @@
+- Start Date: 2023-02-17 14:30
+- RFC PR:
+- ArchGuard Issue: https://github.com/archguard/archguard/issues/97
+
+# Summary
+
+基于 Apache Arrow 格式完善：
+
+- **数据序列化**：在 Client 侧，生成 Arrow 二进制数据，传输到 Server 侧。Server 侧，可直接使用，而无需序列化。
+- **数据分析**：在 Plugin 侧，直接基于 Arrow 格式进行数据分析。
+- **数据传输**：Arrow 比 JSON 更高效，更适合大数据量的传输。
+
+# Basic example
+
+Scanner CLI 输出的数据，可以直接用于 ArchGuard 的数据分析。
+
+## CLI 转换示例
+
+```kotlin
+override fun saveDataStructure(codes: List<CodeDataStruct>) {
+    codes.toDataFrame().writeArrowFeather(File(filename("codes")))
+}
+
+override fun saveApi(apis: List<ContainerService>) {
+    apis.toDataFrame().writeArrowFeather(File(filename("apis")))
+}
+```
+
+## 数据分析示例
+
+如 Kotlin Dataframe
+的示例：[https://kotlin.github.io/dataframe/overview.html](https://kotlin.github.io/dataframe/overview.html)
+
+```kotlin
+val df = DataFrame.read("titanic.csv", delimiter = ';')
+// filter rows
+df.filter { survived && home.endsWith("NY") && age in 10..20 }
+
+// add column
+df.add("birthYear") { 1912 - age }
+
+// sort rows
+df.sortByDesc { age }
+
+// aggregate data
+df.groupBy { pclass }.aggregate {
+    maxBy { age }.name into "oldest person"
+    count { survived } into "survived"
+}
+```
+
+# Motivation
+
+主要原因有：
+
+1. 现有 issue 中：[文件数量多时 OutofMemomery](https://github.com/archguard/archguard/issues/97)
+2. 通过 Arrow 格式，可以直接在 Plugin 侧进行数据分析，而无需序列化。
+
+# Detailed design
+
+针对于 Arrow 引入的改动：
+
+- Scanner CLI 输出的数据，可以直接用于 ArchGuard 的数据分析。
+    - 创建新的 `ArchGuardArrowClient`，用于生成 Arrow 格式的数据。
+- Server 侧，接收到 Arrow 格式的数据，直接使用，而无需序列化。
+    - 创建新的 `ArchGuardArrowServer`，用于接收 Arrow 格式的数据。
+- Plugin 侧，直接基于 Arrow 格式进行数据分析。
+
+针对于 OOM 需要变更的点：
+
+- 在 `SourceCodeAnalyser` 改为 Coroutine，以支持大量数据的分析。
+- 在 Scanner CLI 中，添加流式处理的功能，以支持大量数据的分析。
+- 在 Server 侧，添加流式处理的功能，以支持大量数据的分析。
+
+SourceCodeAnalyser 示例：
+
+```kotlin
+fun analyse(channel: Channel<CodeDataStruct>) = runBlocking {
+    getFilesByPath(context.path) {
+        it.absolutePath.endsWith(".go")
+    }
+        .map { async { analysisByFile(it) } }.awaitAll()
+        .flatten()
+        .forEach { channel.send(it) }
+}
+```
+
+# Drawbacks
+
+修改成本大，需要修改 Scanner CLI、Server、Plugin 等多个模块。
+
+# Alternatives
+
+类似于 Arrow 的格式，还有 Avro、Protobuf 等。
+
+# Adoption strategy
+
+# How we teach this
+
+# Unresolved questions
+
+Kotlin Dataframe 暂时不支持 nested type：https://github.com/Kotlin/dataframe/issues/271

From b7ee27bbf3472afe0ea318678f4310d1eb60e075 Mon Sep 17 00:00:00 2001
From: Fengda Huang <h@phodal.com>
Date: Fri, 17 Feb 2023 17:53:54 +0800
Subject: [PATCH 2/3] add examples with kotlin

---
 text/0004-apache-arrow-as-data-format.md | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/text/0004-apache-arrow-as-data-format.md b/text/0004-apache-arrow-as-data-format.md
index f4d680e..9b7a654 100644
--- a/text/0004-apache-arrow-as-data-format.md
+++ b/text/0004-apache-arrow-as-data-format.md
@@ -49,6 +49,24 @@ df.groupBy { pclass }.aggregate {
 }
 ```
 
+基于 Chapi 的示例：
+
+```kotlin
+val ds = listOf(
+    CodeDataStruct(
+        NodeName = "Main",
+        Functions = listOf(CodeFunction(Name = "main")),
+    )
+)
+
+val dataframe = ds.toDataFrame()
+
+dataframe
+    .filter { it[CodeDataStruct::Functions].isNotEmpty() }
+    .filter { it[CodeDataStruct::Functions].any { function -> function.Name == "main" } }
+    .print()
+```
+
 # Motivation
 
 主要原因有：

From 48707cdb006800280a9a59aacf6119ae2c4338b4 Mon Sep 17 00:00:00 2001
From: Fengda Huang <h@phodal.com>
Date: Fri, 17 Feb 2023 17:57:38 +0800
Subject: [PATCH 3/3] Update 0004-apache-arrow-as-data-format.md

---
 text/0004-apache-arrow-as-data-format.md | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/text/0004-apache-arrow-as-data-format.md b/text/0004-apache-arrow-as-data-format.md
index 9b7a654..9bf9a78 100644
--- a/text/0004-apache-arrow-as-data-format.md
+++ b/text/0004-apache-arrow-as-data-format.md
@@ -105,7 +105,14 @@ fun analyse(channel: Channel<CodeDataStruct>) = runBlocking {
 
 # Drawbacks
 
-修改成本大，需要修改 Scanner CLI、Server、Plugin 等多个模块。
+需要注意：
+
+1. 添加特性需要修改 Scanner CLI、Server、Plugin 等多个模块。
+2. 当使用 Kotlin Dataframe 时，还需要注意以下几点：
+    - 需要将数据类声明为 data class，以便 Dataframe 可以识别数据类的属性。
+    - 集合类型应该是 List、Map 或 ArrayList，而不是 Array，因为 Dataframe 对于集合的处理是基于 Java Collections Framework 的，而不是基于原生数组。
+    - 数据类的属性类型应该是 Kotlin 原生类型或者是具有无参构造函数的类，这是因为 Dataframe 可以直接将这些类型映射到 Arrow 数据类型，而无需进行其他转换。
+    - 如果数据类的属性类型是一个复合类型，比如一个列表或一个嵌套的数据类，需要使用 Arrow 的数据类型来定义这个属性的类型。
 
 # Alternatives