[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.

[github-actions]

This issue has been marked as stale because it has had no activity in the past 365 days. Please remove the stale label or comment below, or this issue will be closed in 14 days. If this improvement is still desired but has no current owner, please add the 'Status: needs champion' label.

[raulcd]

I am removing the assignee in case someone looking for a good-first-issue wants to tackle this one.