v6: OffsetDateTime is serialised with toString(), which drops the seconds and is not RFC 3339
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 68/100
Direzione di ricerca
Individua Filter$DateOperand, InsertManyRequest.marshalValue e DateUtil$CustomTypeAdapterFactory$1, che sono i tre punti di ingresso della serializzazione descritti nell’issue. Riproduci i casi di timestamp arrotondati al minuto per gli inserimenti REST, i batch gRPC e i filtri per data, quindi verifica che tutti e tre i percorsi emettano timestamp RFC 3339 con i secondi e che le scritture e i filtri li accettino.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
The client serialises OffsetDateTime with toString(), which omits the seconds when they are zero. 2024-03-01T00:00:00Z therefore goes on the wire as 2024-03-01T00:00Z, which is not RFC 3339 — partial-time requires hour ":" minute ":" second — and Weaviate rejects it.
Every timestamp on an exact minute boundary is affected, which is most timestamps written by hand. It breaks writes and filters alike.
Reproduction
OffsetDateTime round = OffsetDateTime.parse("2024-03-01T00:00:00Z");
OffsetDateTime withSeconds = OffsetDateTime.parse("2024-03-01T00:00:01Z");
System.out.println(round); // 2024-03-01T00:00Z <- the seconds are gone
System.out.println(withSeconds); // 2024-03-01T00:00:01Z
Against a collection with a single date property when:
FAIL REST insert round -> invalid date property 'when' on class 'DBeaverDateProbe':
requires a string with a RFC3339 formatted date,
but the given value is '2024-03-01T00:00Z'
OK REST insert withSeconds
FAIL gRPC batch round -> invalid date property 'when' on class 'DBeaverDateProbe':
requires a string with a RFC3339 formatted date,
but the given value is '2024-03-01T00:00Z'
OK gRPC batch withSeconds
and on the query side:
FAIL Filter.property("when").gt(round) -> trying parse time as RFC3339 string:
parsing time "2024-03-01T00:00Z" as
"2006-01-02T15:04:05Z07:00": cannot parse "Z" as ":"
OK Filter.property("when").gt(withSeconds) -> 2 rows
OK Filter.property("when").gt("2024-03-01T00:00:00Z") // String overload -> 2 rows
FAIL Filter.createdAt().gt(round) -> same parse error
OK Filter.createdAt().gt(withSeconds)
The only difference between each passing and failing pair is whether the second happens to be zero, which isolates the cause.
Where it comes from
Disassembling every non-protocol class in client6-6.3.1-all.jar and grepping for java/time/OffsetDateTime.toString gives three call sites, one per path:
| class | path | reached by |
|---|---|---|
Filter$DateOperand |
gRPC search | every date filter, plus Filter.createdAt() / Filter.lastUpdatedAt() |
InsertManyRequest.marshalValue |
gRPC batch | data.insertMany(...) — Value.newBuilder().setStringValue(dt.toString()) |
DateUtil$CustomTypeAdapterFactory$1 |
REST / Gson | data.insert(...) and object reads — JsonWriter.value(dt.toString()) |
OffsetDateTime.toString() delegates down to LocalTime.toString(), which emits HH:mm when both second and nano are zero. It is a display format, not a wire format.
Suggested fix
Format explicitly at all three sites instead of calling toString() — for example DateTimeFormatter.ofPattern("yyyy-MM-dd'T'HH:mm:ssXXX"), or ISO_OFFSET_DATE_TIME applied to a value normalised to at least second precision, so the seconds are always present.
Workaround, and where there isn't one
For filters on a property, pass the RFC3339 String rather than an OffsetDateTime: Filter.property(p).gt("2024-03-01T00:00:00Z") takes the String overload and works (shown above).
There is no workaround on the metadata path — Filter.createdAt() and Filter.lastUpdatedAt() return DateProperty, whose comparison methods accept OffsetDateTime only. Same for writes, where the property map value has to be an OffsetDateTime to be recognised as a date at all.
Version
- java-client 6.3.1
- Weaviate 1.39.0
- Lingua principale
- Java
- Stelle
- 34
- Fork
- 29
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di weaviate/java-client
-
v6: ShardReplica.shardName has no @SerializedName, so the shard is always nullForse già presa @dudanogueira l’ha presa 26 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
weaviate/java-client#621 ·
-
v6: Shard.vectorQueueLenght is misspelled, so the vector queue length is always 0Forse già presa @dudanogueira l’ha presa 26 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
weaviate/java-client#619 ·
-
v6: tenant updates are not split at the server's 100-tenant limit, so activate/deactivate fails above 100Forse già presa @dudanogueira l’ha presa 38 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
weaviate/java-client#615 ·
-
v6: NvidiaReranker sends "baseUrl"; the module reads "baseURL"Forse già presa @dudanogueira l’ha presa 41 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
weaviate/java-client#607 ·
-
v6: rerank cannot be used with BM25, Hybrid or FetchObjectsForse già presa @dudanogueira l’ha presa 42 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
weaviate/java-client#603 ·
Tutte le issue di weaviate/java-client
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
-
Make branch and label autocomplete matching locale-independentForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
jenkinsci/gitlab-plugin#1950 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
commons-app/apps-android-commons#6984 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
It's not necessary to copy the memory block in the readWrite() of org.h2.store.fs.mem.FileMemDataAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
h2database/h2database#4435 ·
I maintainer di solito rispondono entro 1 giorno