[Python] Use f-string instead of `string.format`

[kou]

Describe the enhancement requested

For example:

arrow/python/pyarrow/array.pxi

Lines 932 to 934 in a94f860

	raise TypeError("Do not call {}'s constructor directly, use one of "
	"the `pyarrow.Array.from_*` functions instead."
	.format(self.__class__.__name__))

should be:

        raise TypeError(f"Do not call {self.__class__.__name__}'s constructor "
                        "directly, use one of the `pyarrow.Array.from_*` "
                        "functions instead.")

Component(s)

Python

[chilin0525]

take

[chilin0525]

Hi @kou , I'm still working on the PR to refactor all instances of string.format. However, in some cases, I'm unsure whether they should also be refactored.

For example, in the following case, string.format allows passing parameters, making the code more reusable. In such scenarios, f-strings may not be as convenient. Given this, do you think we should still replace them with f-string, or should we leave them as they are? Thanks

arrow/python/pyarrow/parquet/core.py

Lines 1624 to 1695 in 0fbf982

	_read_table_docstring = """
	{0}
	Parameters
	----------
	source : str, pyarrow.NativeFile, or file-like object
	If a string passed, can be a single file name or directory name. For
	file-like objects, only read a single file. Use pyarrow.BufferReader to
	read a file contained in a bytes or buffer-like object.
	columns : list
	If not None, only these columns will be read from the file. A column
	name may be a prefix of a nested field, e.g. 'a' will select 'a.b',
	'a.c', and 'a.d.e'. If empty, no columns will be read. Note
	that the table will still have the correct num_rows set despite having
	no columns.
	use_threads : bool, default True
	Perform multi-threaded column reads.
	schema : Schema, optional
	Optionally provide the Schema for the parquet dataset, in which case it
	will not be inferred from the source.
	{1}
	filesystem : FileSystem, default None
	If nothing passed, will be inferred based on path.
	Path will try to be found in the local on-disk filesystem otherwise
	it will be parsed as an URI to determine the filesystem.
	filters : pyarrow.compute.Expression or List[Tuple] or List[List[Tuple]], default None
	Rows which do not match the filter predicate will be removed from scanned
	data. Partition keys embedded in a nested directory structure will be
	exploited to avoid loading files at all if they contain no matching rows.
	Within-file level filtering and different partitioning schemes are supported.
	{3}
	use_legacy_dataset : bool, optional
	Deprecated and has no effect from PyArrow version 15.0.0.
	ignore_prefixes : list, optional
	Files matching any of these prefixes will be ignored by the
	discovery process.
	This is matched to the basename of a path.
	By default this is ['.', '_'].
	Note that discovery happens only if a directory is passed as source.
	pre_buffer : bool, default True
	Coalesce and issue file reads in parallel to improve performance on
	high-latency filesystems (e.g. S3). If True, Arrow will use a
	background I/O thread pool. If using a filesystem layer that itself
	performs readahead (e.g. fsspec's S3FS), disable readahead for best
	results.
	coerce_int96_timestamp_unit : str, default None
	Cast timestamps that are stored in INT96 format to a particular
	resolution (e.g. 'ms'). Setting to None is equivalent to 'ns'
	and therefore INT96 timestamps will be inferred as timestamps
	in nanoseconds.
	decryption_properties : FileDecryptionProperties or None
	File-level decryption properties.
	The decryption properties can be created using
	``CryptoFactory.file_decryption_properties()``.
	thrift_string_size_limit : int, default None
	If not None, override the maximum total string size allocated
	when decoding Thrift structures. The default limit should be
	sufficient for most Parquet files.
	thrift_container_size_limit : int, default None
	If not None, override the maximum total size of containers allocated
	when decoding Thrift structures. The default limit should be
	sufficient for most Parquet files.
	page_checksum_verification : bool, default False
	If True, verify the checksum for each page read from the file.
	Returns
	-------
	{2}
	{4}
	"""

[kou]

We don't need to use f-string for it. If we use f-string for it, we can't reuse the template in multiple methods.

[raulcd]

Issue resolved by pull request 45629
#45629