chilin0525 commented on code in PR #45629:
URL: https://github.com/apache/arrow/pull/45629#discussion_r2030017862
##########
python/pyarrow/feather.py:
##########
@@ -69,10 +69,8 @@ def read_table(self, columns=None):
def validate_schemas(self, piece, table):
if not self.schema.equals(table.schema):
- raise ValueError('Schema in {!s} was different. \n'
- '{!s}\n\nvs\n\n{!s}'
- .format(piece, self.schema,
- table.schema))
+ raise ValueError(f'Schema in {piece!s} was different. \n'
+ f'{self.schema!s}\n\nvs\n\n{table.schema!s}')
Review Comment:
Resolved in
[c926dde](https://github.com/apache/arrow/pull/45629/commits/c926ddec73bfdeb7d6c3b3768ade16f8146eadd4)
##########
python/pyarrow/pandas_compat.py:
##########
@@ -621,13 +620,12 @@ def convert_column(col, field):
except (pa.ArrowInvalid,
pa.ArrowNotImplementedError,
pa.ArrowTypeError) as e:
- e.args += ("Conversion failed for column {!s} with type {!s}"
- .format(col.name, col.dtype),)
+ e.args += (
+ f"Conversion failed for column {col.name!s} with type
{col.dtype!s}",)
Review Comment:
Resolved in
[c926dde](https://github.com/apache/arrow/pull/45629/commits/c926ddec73bfdeb7d6c3b3768ade16f8146eadd4)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
