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

[C++] Simplify FileFormat classes to singletons

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Won't Do
    • None
    • 0.16.0
    • C++

    Description

      ParquetFileFormat has no state, so passing it around by shared_ptr<FileFormat> is not necessary; we could just keep a single static instance and pass raw pointers.

      wesmckinn is there a case where a FileFormat might have state?

      Attachments

        Activity

          People

            bkietz Ben Kietzman
            bkietz Ben Kietzman
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: