apache / apache/datafusion-comet
virtual_hosted_style_request bad calculation
- Lingua principale
- Scala
- Stelle
- 1.3k
- Fork
- 375
- Merge medio
- 2g 10h
- PR unite (30g)
- 231
Descrizione
### Describe the bug
while setting the connection to S3 in (native/core/src/parquet/objectstore/s3.rs)
a mistake is being made in calculating `virtual_hosted_style_request`
which causes the connection to use a combination of path style and virtual hosted style.
```
virtual_hosted_style_request = path_style.to_lowercase() == "true";
s3_configs.insert(
AmazonS3ConfigKey::VirtualHostedStyleRequest,
virtual_hosted_style_request.to_string(),
);
```
```
if virtual_hosted_style_request {
if endpoint.ends_with("/") {
Some(format!("{endpoint}{bucket}"))
} else {
Some(format!("{endpoint}/{bucket}"))
}
} else {
Some(endpoint) // Avoid extra to_string() call since endpoint is already a String
}
```
### Steps to reproduce
_No response_
### Expected behavior
_No response_
### Additional context
_No response_
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Inizia da native/core/src/parquet/objectstore/s3.rs e segui come path_style viene convertito in virtual_hosted_style_request e poi passato alla configurazione S3. Verifica che la gestione di endpoint e bucket utilizzi in modo coerente un solo stile di indirizzamento; il lavoro è completo quando la connessione non combina più richieste path-style e virtual-hosted-style.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- aws, rust
- Ambito
- cloud
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 55/100