Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Data] Fix logging output from
write_xxx
APIs (#48096)
## Why are these changes needed? Followup to #47942. Fixes the formatting of the output logged after write operation finishes, which incorrectly printed out the entire `dataclasses.field` object. Previous logging: ``` 2024-10-17 17:26:48,396 INFO datasink.py:103 -- Write operation succeeded. Aggregated write results: Field(name='num_rows',type=<class 'int'>,default=0,default_factory=<dataclasses._MISSING_TYPE object at 0x104ad5970>,init=True,repr=True,hash=None,compare=True,metadata=mappingproxy({}),_field_type=_FIELD): 100 Field(name='size_bytes',type=<class 'int'>,default=0,default_factory=<dataclasses._MISSING_TYPE object at 0x104ad5970>,init=True,repr=True,hash=None,compare=True,metadata=mappingproxy({}),_field_type=_FIELD): 800 ``` Fixed logging: ``` 2024-10-17 17:30:52,491 INFO datasink.py:103 -- Write operation succeeded. Aggregated write results: - num_rows: 100 - size_bytes: 800 ``` ## Related issue number ## Checks - [x] I've signed off every commit(by using the -s flag, i.e., `git commit -s`) in this PR. - [x] I've run `scripts/format.sh` to lint the changes in this PR. - [ ] I've included any doc changes needed for https://docs.ray.io/en/master/. - [ ] I've added any new APIs to the API Reference. For example, if I added a method in Tune, I've added it in `doc/source/tune/api/` under the corresponding `.rst` file. - [x] I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/ - Testing Strategy - [x] Unit tests - [ ] Release tests - [ ] This PR is not tested :( Signed-off-by: Scott Lee <[email protected]>
- Loading branch information