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

http: add parameters commitLag and maxUncommittedRows to /imp #1203

Merged
merged 20 commits into from
Jul 22, 2021

Conversation

marregui
Copy link
Contributor

Came as a side effect of #1181.

The solution to that issue was to partition by DAY, or to buy more ram.
Having more control over these two parameters is nice to have.

@marregui marregui self-assigned this Jul 16, 2021
@marregui marregui added the REST API Issues or changes relating to the HTTP endpoints label Jul 16, 2021
@marregui marregui added this to In progress in HTTP / REST API via automation Jul 16, 2021
@marregui marregui force-pushed the ma/issue-1181-http-additional-params branch 2 times, most recently from e1f365c to d620dd4 Compare July 20, 2021 17:33
@marregui marregui requested a review from ideoma July 20, 2021 18:07
@marregui marregui marked this pull request as ready for review July 20, 2021 18:09
@marregui marregui force-pushed the ma/issue-1181-http-additional-params branch from 5eed1f6 to 20bb1ff Compare July 21, 2021 21:14
@marregui marregui requested a review from ideoma July 22, 2021 08:06
@ideoma
Copy link
Collaborator

ideoma commented Jul 22, 2021

[PR Coverage check]

😍 pass : 48 / 48 (100.00%)

file detail

path covered line new line coverage
🔵 io/questdb/cairo/TableWriter.java 1 1 100.00%
🔵 io/questdb/cutlass/http/processors/TextImportProcessor.java 16 16 100.00%
🔵 io/questdb/cutlass/text/CairoTextWriter.java 23 23 100.00%
🔵 io/questdb/griffin/FunctionParser.java 1 1 100.00%
🔵 io/questdb/cairo/TableReader.java 2 2 100.00%
🔵 io/questdb/cutlass/line/tcp/LineTcpMeasurementScheduler.java 1 1 100.00%
🔵 io/questdb/cutlass/text/TextLoader.java 4 4 100.00%

@bluestreak01 bluestreak01 merged commit 8f2f98f into master Jul 22, 2021
HTTP / REST API automation moved this from In progress to Done Jul 22, 2021
@bluestreak01 bluestreak01 deleted the ma/issue-1181-http-additional-params branch July 22, 2021 10:15
@ideoma ideoma added this to the 6.0.5 milestone Jul 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
REST API Issues or changes relating to the HTTP endpoints
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants