Skip to content

KAFKA-2587:Increasing timeout for the test verification.#260

Closed
Parth-Brahmbhatt wants to merge 1 commit into
apache:trunkfrom
Parth-Brahmbhatt:KAFKA-2587
Closed

KAFKA-2587:Increasing timeout for the test verification.#260
Parth-Brahmbhatt wants to merge 1 commit into
apache:trunkfrom
Parth-Brahmbhatt:KAFKA-2587

Conversation

@Parth-Brahmbhatt

Copy link
Copy Markdown
Contributor

No description provided.

@asfbot

asfbot commented Sep 29, 2015

Copy link
Copy Markdown

kafka-trunk-git-pr #596 FAILURE
Looks like there's a problem with this pull request

@ijuma

ijuma commented Sep 29, 2015

Copy link
Copy Markdown
Member

LGTM

@ijuma

ijuma commented Sep 29, 2015

Copy link
Copy Markdown
Member

@junrao can you please take a look? The test that was added as part of the simple authorization work has been a bit unstable and Parth increased the time out and improved the failure message to help us understand future failures better.

@harshach

Copy link
Copy Markdown

+1

@asfgit asfgit closed this in 6b1a92c Sep 29, 2015
@ijuma

ijuma commented Sep 29, 2015

Copy link
Copy Markdown
Member

Thanks for merging @harshach and congratulations on your first merge to trunk. :)

jsancio pushed a commit to jsancio/kafka that referenced this pull request Aug 6, 2019
* CONFLUENT: Bump cc-base version.

* Gradle artifacts are unsigned; pass apt flag.

* Point to updated cc-base; remove flags
soondenana pushed a commit to soondenana/kafka that referenced this pull request Mar 4, 2020
wyuka pushed a commit to wyuka/kafka that referenced this pull request Feb 4, 2022
TICKET =
LI_DESCRIPTION =
Currently, the CI workflows are triggered when there is a push event, e.g.
when a PR is merged. Running the workflows upon push events is not required
and thus wastes computation power. This PR removes the push event trigger.
EXIT_CRITERIA =
davide-armand pushed a commit to aiven/kafka that referenced this pull request Dec 1, 2025
…s returned (apache#260)

* INK-215: Return invalid produce results if no other batch succeeds

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: spotless

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: add metrics

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: revert change to integration test

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: license in new file

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: test failures

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: spotless

Signed-off-by: Greg Harris <greg.harris@aiven.io>

---------

Signed-off-by: Greg Harris <greg.harris@aiven.io>
jeqo pushed a commit to aiven/kafka that referenced this pull request Jan 16, 2026
…s returned (apache#260)

* INK-215: Return invalid produce results if no other batch succeeds

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: spotless

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: add metrics

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: revert change to integration test

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: license in new file

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: test failures

Signed-off-by: Greg Harris <greg.harris@aiven.io>

* fixup: spotless

Signed-off-by: Greg Harris <greg.harris@aiven.io>

---------

Signed-off-by: Greg Harris <greg.harris@aiven.io>
traceyyoshima added a commit to traceyyoshima/kafka that referenced this pull request May 17, 2026
…pache#260)

Refreshed snapshot. Eight language-engine PRs landed this session:

- PR apache#253 — Annotation paren anchor for continuation slots
- PR apache#254 — StructuralBody2x catalog class for inline lambda bodies
- PR apache#255 — Lambda body as continuation enclosing scope
- PR apache#256 — Inline-lambda body close brace catalog
- PR apache#257 — emittedLineStarts restricted to OWNER's prefix slot
- PR apache#258 — alignmentTargets restricted to OWNER's prefix slot
- PR apache#259 — Chain wrap preservation when siblings agree on column
- PR apache#260 — Binary OPERATOR_PREFIX consults outermost binary's anchor

Sweep stats (post-apache#260):
  files scanned : 5,834
  files changed : 989 (17.0%)     [initial PR #6: 1,038]
  byte delta    : -10,663
  reformat time : ~7s total

Almost all called-out files from the user's original review now
fully closed. Remaining residuals (<5 files) are correct
normalisations of slightly non-standard source or trailing
whitespace inside comment content (intentionally out of scope for
this engine version per user design choice).
traceyyoshima added a commit to traceyyoshima/kafka that referenced this pull request May 17, 2026
…pache#261)

Refreshed snapshot. Nine language-engine PRs landed this session:

- PR apache#253 — Annotation paren anchor for continuation slots
- PR apache#254 — StructuralBody2x catalog class for inline lambda bodies
- PR apache#255 — Lambda body as continuation enclosing scope
- PR apache#256 — Inline-lambda body close brace catalog
- PR apache#257 — emittedLineStarts restricted to OWNER's prefix slot
- PR apache#258 — alignmentTargets restricted to OWNER's prefix slot
- PR apache#259 — Chain wrap preservation when siblings agree on column
- PR apache#260 — Binary OPERATOR_PREFIX consults outermost binary's anchor
- PR apache#261 — Nested continuation slots use innermost enclosing scope

Sweep stats (post-apache#261):
  files scanned : 5,834
  files changed : 986 (16.9%)     [initial PR #6: 1,038]
  byte delta    : -11,137          [initial PR #6: -6,661]
  reformat time : ~7s total

PR #7 review comments status:
  - Comments 1+2 closed (PR apache#261)
  - Comments 3 + 4/5 deferred (workbench-first investigation; diagnosis
    captured in language-engine memory project_pr7_deferred_comments_2026_05_17.md)
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

Successfully merging this pull request may close these issues.

4 participants