corgy-w opened a new pull request, #11996:
URL: https://github.com/apache/seatunnel/pull/11996

   ### Purpose of this pull request
   
   Replace `list.size() > 0` / `list.size() == 0` collection checks with 
`!isEmpty()` / `isEmpty()` in the Easysearch and Elasticsearch connectors (sink 
flush, scroll reader, index serializer factory). Behavior is unchanged, but 
`isEmpty()` expresses the intent directly and avoids deriving the size of an 
already-computed collection. This matches the Java collection idiom used 
elsewhere in the codebase.
   
   Files:
   - `connector-easysearch`: `EasysearchSourceReader`, `EasysearchSinkWriter`, 
`IndexSerializerFactory`
   - `connector-elasticsearch`: `ElasticsearchSinkWriter`, 
`IndexSerializerFactory`
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   - `./mvnw spotless:apply -pl 
seatunnel-connectors-v2/connector-easysearch,seatunnel-connectors-v2/connector-elasticsearch`
   - `./mvnw -pl 
seatunnel-connectors-v2/connector-easysearch,seatunnel-connectors-v2/connector-elasticsearch
 compile`


-- 
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]

Reply via email to