apache / apache/datafusion-comet

virtual_hosted_style_request bad calculation

Aperta
#2,802 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
bug priority:medium
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.