Presto: Allow configuring target result size in JDBC

Created on 6 Dec 2019  路  4Comments  路  Source: prestosql/presto

https://github.com/prestosql/presto/issues/2210 adds configuration toggle on the server.

Additionally, we should allow settings this in JDBC via JDBC URI parameters.

enhancement good first issue
Source
picture findepi

Most helpful comment

I am ready to merge this change. Can you assign this to me?

picture sajjoseph on 20 Apr 2020
👍3 ❤2

All 4 comments

I am ready to merge this change. Can you assign this to me?

sajjoseph picture sajjoseph on 20 Apr 2020
👍3 ❤2

@sajjoseph does ur pr include https://github.com/prestosql/presto/pull/3017 ?

tooptoop4 picture tooptoop4 on 20 Apr 2020

It doesn't.
But, I fixed the suggestions listed by聽@kokosing and @Praveen2112 in your PR. Let me know if you want me to have a separate PR for 3017 fixes. I also added the changes for Presto CLI and I will open a separate request for that.
By the way, I don't see significant improvement in performance as the data transfer rate depends on the network connection/speed. We definitely need parallel data retrieval than these small gimmicks.

sajjoseph picture sajjoseph on 20 Apr 2020

@sajjoseph would be great if u can take over 3017 馃

tooptoop4 picture tooptoop4 on 20 Apr 2020
Was this page helpful?
0 / 5 - 0 ratings

Related issues

theoretical-olive picture theoretical-olive  路  5Comments
JamesRTaylor picture JamesRTaylor  路  5Comments
anismiles picture anismiles  路  3Comments
lxynov picture lxynov  路  4Comments
yourmain picture yourmain  路  4Comments