Re: Review Request 67057: TestHiveImport is failing on Jenkins

2018-05-10 Thread Boglarka Egyed

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67057/#review202845
---


Ship it!




Tested on Ubuntu and Mac OS too, both passed. Thanks for the fix Daniel!

- Boglarka Egyed


On May 10, 2018, 12:53 p.m., daniel voros wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67057/
> ---
> 
> (Updated May 10, 2018, 12:53 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-3321
> https://issues.apache.org/jira/browse/SQOOP-3321
> 
> 
> Repository: sqoop-trunk
> 
> 
> Description
> ---
> 
> I believe this is due to case sensitivity of file names in Linux (as opposed 
> to MacOS). The table name gets converted to lowercase when importing but 
> we're referring to it with it's original casing when trying to verify its 
> contents in ParquetReader.
> 
> Tests are passing after converting these three table names to all lowercase 
> in TestHiveImport:
> 
> - APPEND_HIVE_IMPORT_AS_PARQUET
> - NORMAL_HIVE_IMPORT_AS_PARQUET
> - CREATE_OVERWRITE_HIVE_IMPORT_AS_PARQUET
> 
> 
> Diffs
> -
> 
>   src/test/org/apache/sqoop/hive/TestHiveImport.java bc19b697 
> 
> 
> Diff: https://reviews.apache.org/r/67057/diff/1/
> 
> 
> Testing
> ---
> 
> Run TestHiveImport.
> 
> 
> Thanks,
> 
> daniel voros
> 
>



Review Request 67057: TestHiveImport is failing on Jenkins

2018-05-10 Thread daniel voros

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/67057/
---

Review request for Sqoop.


Bugs: SQOOP-3321
https://issues.apache.org/jira/browse/SQOOP-3321


Repository: sqoop-trunk


Description
---

I believe this is due to case sensitivity of file names in Linux (as opposed to 
MacOS). The table name gets converted to lowercase when importing but we're 
referring to it with it's original casing when trying to verify its contents in 
ParquetReader.

Tests are passing after converting these three table names to all lowercase in 
TestHiveImport:

- APPEND_HIVE_IMPORT_AS_PARQUET
- NORMAL_HIVE_IMPORT_AS_PARQUET
- CREATE_OVERWRITE_HIVE_IMPORT_AS_PARQUET


Diffs
-

  src/test/org/apache/sqoop/hive/TestHiveImport.java bc19b697 


Diff: https://reviews.apache.org/r/67057/diff/1/


Testing
---

Run TestHiveImport.


Thanks,

daniel voros