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

HIVE-27502 Enable HiveKafkaProducerTest #4483

Closed
wants to merge 1 commit into from

Conversation

akshat0395
Copy link
Contributor

What changes were proposed in this pull request?

Enable HiveKafkaProducerTest

Why are the changes needed?

HIVE-27502 post downgrade of Kafka Version HIVE-27475.

The flaky test started passing http://ci.hive.apache.org/job/hive-flaky-check/717/ hence enabling the test again

Does this PR introduce any user-facing change?

No

Is the change a dependency upgrade?

No

How was this patch tested?

Existing Unit tests

Copy link

@aturoczy aturoczy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow :) 🥇

@sonarcloud
Copy link

sonarcloud bot commented Jul 15, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Copy link
Contributor

@zabetak zabetak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, will merge soon!

@zabetak zabetak closed this in 8ea1d78 Jul 19, 2023
yeahyung pushed a commit to yeahyung/hive that referenced this pull request Jul 20, 2023
…eviewed by Attila Turoczy, Stamatis Zampetakis)

Closes apache#4483
tarak271 pushed a commit to tarak271/hive-1 that referenced this pull request Dec 19, 2023
…eviewed by Attila Turoczy, Stamatis Zampetakis)

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

Successfully merging this pull request may close these issues.

4 participants