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

HTTP2 Branch error on getproperty #54

Closed
mrufsvold opened this issue Aug 5, 2024 · 1 comment
Closed

HTTP2 Branch error on getproperty #54

mrufsvold opened this issue Aug 5, 2024 · 1 comment

Comments

@mrufsvold
Copy link

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
Copy link
Author

closed by #55

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant