leaves12138 opened a new pull request, #2450:
URL: https://github.com/apache/incubator-paimon/pull/2450
### Purpose
Refactory code about MutableObjectIterator. And remove method:
```code
/**
* Gets the next element from the collection. The contents of that next
element is put into the
* given reuse object, if the type is mutable.
*
* @param reuse The target object into which to place next element if E
is mutable.
* @return The filled object or <code>null</code> if the iterator is
exhausted.
* @throws IOException Thrown, if a problem occurred in the underlying
I/O layer or in the
* serialization / deserialization logic
*/
E next(E reuse) throws IOException;
```
Reason:
*This method is confusing. After code `row = next(row)`, we don't know
exactly what row is point to. Does it stay itself, or something else.
*Every `MutableObjectIterator` deserves its own reuse object, it is simple
to understand and maintain.
### Tests
<!-- List UT and IT cases to verify this change -->
### API and Format
<!-- Does this change affect API or storage format -->
### Documentation
<!-- Does this change introduce a new feature -->
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
