Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-2843

[Format/C++/Java] Removing field layout from Schema.fbs breaks backward compatibility

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: 0.8.0, 0.9.0
    • Fix Version/s: None
    • Component/s: Format

      Description

      Change in ARROW-1785

      Breaks backward compatibility with schema generated in prior versions of Arrow, as VectorLayout field was removed from Field table.

      This is definitely not supported behavior in FlatBuffers.

      I suggest to put back following field

      layout: [ VectorLayout ] (deprecated);
      

       into 

      table Field 
      

      and mark it "deprecated" - as shown above

      It will also require putting back 

      table VectorLayout
      

      which can have all the deprecated fields as well.

      code generation footprint will be minimal in this case, as FlatBuffers won't generate getters for deprecated fields, but it will really solve backward compatibility issues.

      Also may be a good idea to introduce "version" into Schema.

       

       

       

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                yufeldman Yuliya Feldman
                Reporter:
                yufeldman Yuliya Feldman
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 3h 40m
                  3h 40m