trinodb/trino

Make default target result size configurable in ExecutingStatementResource

Open

#2,210 opened on Dec 5, 2019

View on GitHub
 (5 comments) (0 reactions) (0 assignees)Java (2,678 forks)batch import
enhancementgood first issue

Repository metrics

Stars
 (9,113 stars)
PR merge metrics
 (Avg merge 5d 9h) (306 merged PRs in 30d)

Description

https://github.com/prestodb/presto/issues/11908 mentions targetresultsize, is anyone open to a pr to make it configurable in server side properties? ie so folks can try 4mb instead of 1mb. returning 1gb data into reporting tool with odbc/jdbc is really slow. dumping to file on presto server is fast but not an option because the reporting tool cant access files on presto server

Contributor guide

Research direction
Add a configuration property to set the default target result size in ExecutingStatementResource, similar to how other properties are defined in the server side configuration.
Tech stack
java
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitJava
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75