[C++][Parquet] Add arrow::Result version of parquet::arrow::FileReader::Make()

[raulcd]

Describe the enhancement requested

The current version uses arrow::Status and std::unique_ptr<FileReader>*:

arrow/cpp/src/parquet/arrow/reader.h

Lines 119 to 127 in ae497bf

	static ::arrow::Status Make(::arrow::MemoryPool* pool,
	std::unique_ptr<ParquetFileReader> reader,
	const ArrowReaderProperties& properties,
	std::unique_ptr<FileReader>* out);
	/// Factory function to create a FileReader from a ParquetFileReader
	static ::arrow::Status Make(::arrow::MemoryPool* pool,
	std::unique_ptr<ParquetFileReader> reader,
	std::unique_ptr<FileReader>* out);

Component(s)

C++, Parquet

[raulcd]

@kou @pitrou from my understanding those are things that we want to keep doing and migrating to ::arrow::Result, right? I could do some of those :)

[pitrou]

Yes, we might have let slip a couple of them.

You could definitely do some of those, but it could also be an easy task for a first-time contributor.

[malinjawi]

Hey @raulcd @pitrou I would be happy to work on this issue. I just got my first PR merged and happy to continue making contributions to Arrow.

[raulcd]

Thanks @malinjawi for your contributions!

[malinjawi]

Hey @raulcd @pitrou thanks for raising this issue. I have raised a PR and addressed other related changes found with the current versions of arrow::Status and std::unique_ptr<FileReader>*. Please let me know i there are anything else I can address to solve this issue.