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

Updated status of 2 tests to accepted in pr-data.csv #1567

Merged

Conversation

nikunjagarwal321
Copy link
Contributor

No description provided.

@darko-marinov
Copy link
Contributor

Great. Can you check more tests in hive?

@darko-marinov darko-marinov merged commit bbdcda2 into TestingResearchIllinois:main Dec 5, 2024
1 check passed
@nikunjagarwal321
Copy link
Contributor Author

Sure Professor, there is another test whose status can be updated. Found recently that the PR was raised last year but hasn't been merged yet, but this test fixes that test. Can I go ahead and update the status of that test as well?

org.apache.hadoop.hive.serde2.columnar.TestLazyBinaryColumnarSerDe.testSerDe

@darko-marinov
Copy link
Contributor

You can open a new PR but provide enough evidence to convince @Ajitesh27 that the commit you claim indeed fixes the test.

@nikunjagarwal321
Copy link
Contributor Author

Sure!

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.

2 participants