[
https://issues.apache.org/jira/browse/SQOOP-1836?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14247219#comment-14247219
]
Sqoop QA bot commented on SQOOP-1836:
-------------------------------------
Testing file
[SQOOP-1836.patch|https://issues.apache.org/jira/secure/attachment/12687297/SQOOP-1836.patch]
against branch sqoop2 took 0:26:19.240959.
{color:red}Overall:{color} -1 due to 3 errors
{color:red}ERROR:{color} Patch does not add/modifny any test case
{color:red}ERROR:{color} Some integration tests failed
{color:red}ERROR:{color} Failed integration test:
{{org.apache.sqoop.integration.repository.derby.upgrade.Derby1_99_4UpgradeTest}}
{color:green}SUCCESS:{color} Clean was successful
{color:green}SUCCESS:{color} Patch applied correctly
{color:green}SUCCESS:{color} License check passed
{color:green}SUCCESS:{color} Patch compiled
{color:green}SUCCESS:{color} All unit tests passed
Console output is available
[here|https://builds.apache.org/job/PreCommit-SQOOP-Build/783/console].
This message is automatically generated.
> Convert the List in Schema Columns to Array for better lookup time
> ------------------------------------------------------------------
>
> Key: SQOOP-1836
> URL: https://issues.apache.org/jira/browse/SQOOP-1836
> Project: Sqoop
> Issue Type: Bug
> Components: sqoop2-framework
> Reporter: Veena Basavaraj
> Assignee: Veena Basavaraj
> Labels: newbie
> Fix For: 1.99.5
>
> Attachments: SQOOP-1836.patch
>
>
> see comments in the https://reviews.apache.org/r/28636/
> Most times we will need to index the column array when extracting and
> loading. So it is good perf improvement to store it as array as per [~jarcec]
> https://github.com/apache/sqoop/blob/sqoop2/common/src/main/java/org/apache/sqoop/schema/Schema.java#L62
> Clean up all places where toArray is done and move this to this to the Schema
> class itself.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
