Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2829#discussion_r226218717
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -206,6 +206,8 @@ struct FileFooter3{
4: optional list<BlockletInfo3> blocklet_info_list3; // Information
about blocklets of all columns in this file for V3 format
5: optional dictionary.ColumnDictionaryChunk dictionary; // Blocklet
local dictionary
6: optional bool is_sort; // True if the data is sorted in this file,
it is used for compaction to decide whether to use merge sort or not
+ 7: optional string written_by; // written by is used to write who
wrote the file, it can be LOAD, or SDK etc
--- End diff --
please add a map<string, string> property, and put these two new fields
into this property map. We can extend any field by using this property map
instead of adding more fields
---
