JuliaServices / CloudStore.jl

A simple, consistent, and performant API for interacting with common cloud storage abstractions
Other
16 stars 8 forks source link

HTTP2 Branch error on `getproperty` #54

Closed mrufsvold closed 1 month ago

mrufsvold commented 1 month ago

Having set my HTTP.jl version to 1.10.3 (#53 ), I tried to use the jq-http2 branch to pull an object from S3. Unfortunately, I'm hitting this error:

import CloudStore
config = CloudStore.AWS.AWSCredentials()
url = "https://ppad-processing-processed-datanode-output-prod.s3.amazonaws.com/rollout2024q3-197/rollout2024q3-197_innetwork/data_version_p=1.1.0/versioneddatasourceid_p=197/sourcefileid_p=1605809/06c4b233-fa63-4bd4-b053-37b7a80aae35.parquet"
region = "us-east-1"
CloudStore.get(
    url
    ;
    service="S3",
    region=region,
    credentials = config
)
ERROR: TaskFailedException

    nested task error: type Response has no field metrics
    Stacktrace:
     [1] getproperty
       @ .\Base.jl:37 [inlined]
     [2] (::CloudStore.API.var"#26#28"{…})()
       @ CloudStore.API C:\Users\mrufsvold\.julia\packages\CloudStore\qT25C\src\get.jl:229
Stacktrace:
 [1] sync_end(c::Channel{Any})
   @ Base .\task.jl:448       
 [2] macro expansion
   @ .\task.jl:480 [inlined]
 [3] getObjectImpl(x::CloudBase.AWS.Bucket, key::String, out::Nothing; multipartThreshold::Int64, partSize::Int64, batchSize::Int64, allowMultipart::Bool, objectMaxSize::Nothing, decompress::Bool, zlibng::Bool, headers::Vector{…}, lograte::Bool, kw::@Kwargs{…})
   @ CloudStore.API C:\Users\mrufsvold\.julia\packages\CloudStore\qT25C\src\get.jl:214
 [4] getObjectImpl
   @ C:\Users\mrufsvold\.julia\packages\CloudStore\qT25C\src\get.jl:98 [inlined]
 [5] get
   @ C:\Users\mrufsvold\.julia\packages\CloudStore\qT25C\src\s3.jl:33 [inlined]
 [6] get
   @ C:\Users\mrufsvold\.julia\packages\CloudStore\qT25C\src\CloudStore.jl:65 [inlined]
 [7] get(url::String, out::Nothing; region::String, nowarn::Bool, kw::@Kwargs{…})
   @ CloudStore C:\Users\mrufsvold\.julia\packages\CloudStore\qT25C\src\CloudStore.jl:78
 [8] get
   @ C:\Users\mrufsvold\.julia\packages\CloudStore\qT25C\src\CloudStore.jl:76 [inlined]
 [9] top-level scope
   @ c:\Users\mrufsvold\Projects\DIL-price-transparency-psd\TableConsolidator.jl\playground.jl:116
Some type information was truncated. Use `show(err)` to see complete types.

It appears that the last commit introduced lines that reference a metrics property off the response object, but in this case, it's getting an HTTP.Response object, which doesn't have such a property?

mrufsvold commented 1 month ago

closed by #55