Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

s2geometry: ERROR Invalid S2LatLng in S2LatLng::ToPoint #27090

Closed
qoega opened this issue Aug 2, 2021 · 3 comments · Fixed by #52260
Closed

s2geometry: ERROR Invalid S2LatLng in S2LatLng::ToPoint #27090

qoega opened this issue Aug 2, 2021 · 3 comments · Fixed by #52260
Labels
fuzz Problem found by one of the fuzzers

Comments

@qoega
Copy link
Member

qoega commented Aug 2, 2021

https://clickhouse-test-reports.s3.yandex.net/0/6951e8147d7a25f7bb4f7d43738793cff239db6d/stress_test_(debug).html#fail1

:) SELECT geoToS2(toFloat64(toUInt64(-1)), toFloat64(toUInt64(-1)));

SELECT geoToS2(toFloat64(toUInt64(-1)), toFloat64(toUInt64(-1)))

Query id: f1ef3f1d-b46d-4ef4-a5e5-869798d8379c

┌─geoToS2(toFloat64(toUInt64(-1)), toFloat64(toUInt64(-1)))─┐
│                                       4864204703484167331 │
└───────────────────────────────────────────────────────────┘
$ grep -Fa ERROR stderr.log 
../contrib/s2geometry/src/s2/s2latlng.cc:37 ERROR Invalid S2LatLng in S2LatLng::ToPoint: [1.84467e+19, 1.84467e+19]

Maybe it should throw exception if point is invalid. Also it should not print errors to stderr.

@qoega qoega added the fuzz Problem found by one of the fuzzers label Aug 2, 2021
@tavplubix
Copy link
Member

Illegal type Array(Nullable(Nothing)) of first argument of function parseDateTime32BestEffortOrNull. Conversion functions with postfix 'OrZero' or 'OrNull' should take String argument: While processing parseDateTime32BestEffortOrNull([NULL], [NULL]) - looks like expected behavior

Cannot add message to the log caused by MEMORY_LIMIT_EXCEEDED exception on attempt to write message to log

Assertion in DatabaseReplicated - duplicate of #27011

@tavplubix tavplubix changed the title Illegal type Array(Nullable(Nothing)) of first argument of function parseDateTime32BestEffortOrNull s2geometry: ERROR Invalid S2LatLng in S2LatLng::ToPoint Aug 2, 2021
@nikitamikhaylov nikitamikhaylov self-assigned this Aug 2, 2021
@qoega
Copy link
Member Author

qoega commented Jun 17, 2022

Same lib. Other place.

../contrib/s2geometry/src/s2/s2latlng_rect.h:382 ERROR Invalid rect: [50.4210420, 0.4931979], [2.1000000, 2.9000000]

@tavplubix
Copy link
Member

https://s3.amazonaws.com/clickhouse-test-reports/51502/bbd4159beeca878616546b77a3dd36b3a9a391d6/stateless_tests__debug__[4_5]/stderr.log

/build/contrib/s2geometry/src/s2/s2latlng_rect.h:382 ERROR Invalid rect: [50.4210420, 0.4931979], [2.1000000, 2.9000000]
/build/contrib/s2geometry/src/s2/s2latlng.cc:38 ERROR Invalid S2LatLng in S2LatLng::ToPoint: [1.84467e+19, 1.84467e+19]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fuzz Problem found by one of the fuzzers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants