Kontinuation commented on code in PR #2305:
URL: https://github.com/apache/sedona/pull/2305#discussion_r2289619672
##########
common/src/main/java/org/apache/sedona/common/S2Geography/PolylineGeography.java:
##########
@@ -138,22 +138,26 @@ public static PolylineGeography decode(UnsafeInput in,
EncodeTag tag) throws IOE
// 2) Skip past any covering cell-IDs written by encodeTagged
tag.skipCovering(in);
- // 3) Ensure we have at least 4 bytes for the count
- if (in.available() < Integer.BYTES) {
- throw new IOException("PolylineGeography.decodeTagged error:
insufficient header bytes");
- }
-
- // 5) Read the number of polylines (4-byte)
- int count = in.readInt();
+ // 3) Read the number of polylines (4-byte)
+ try {
+ int count = in.readInt();
Review Comment:
Copilot is probably misinterpreting the diff, we can ignore it.
--
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]
