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

AMBARI-22999 : Ambari Hive View 2.0 'Upload Table' does not support U… #527

Merged

Conversation

nitirajrathore
Copy link
Contributor

…TF8 files with BOM (nitirajrathore) (#510)

  • AMBARI-22999 : Ambari Hive View 2.0 'Upload Table' does not support UTF8 files with BOM (nitirajrathore)

  • AMBARI-22999 : Added relevant test cases (nitirajrathore)

What changes were proposed in this pull request?

As reviewed in branch-2.6
Using BomInputStream to filterout BOM characters from the file before reading it.

How was this patch tested?

Manual testing done buy uploading the file in hive view 2.0 with BOM characters and checking if files is uploaded and given first row (which contains BOM) is treated as the list of columns for the created table.
Unit test cases added.

…TF8 files with BOM (nitirajrathore) (apache#510)

* AMBARI-22999 : Ambari Hive View 2.0 'Upload Table' does not support UTF8 files with BOM (nitirajrathore)

* AMBARI-22999 : Added relevant test cases (nitirajrathore)
@asfgit
Copy link

asfgit commented Mar 2, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/924/
Test FAILed.
Test FAILured.

@rlevas
Copy link
Contributor

rlevas commented Mar 2, 2018

retest this please

@asfgit
Copy link

asfgit commented Mar 3, 2018

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/937/
Test FAILed.
Test FAILured.

@nitirajrathore
Copy link
Contributor Author

nitirajrathore commented Mar 5, 2018

Build seems to be failing in unrelated code. I am pushing as got approval.

Tests run: 3, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 3.883 sec - in org.apache.ambari.server.alerts.AmbariPerformanceRunnableTest
FATAL: command execution failed
java.io.EOFException
at java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2671)
at java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:3146)

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.

3 participants