[jira] [Commented] (SQOOP-2682) Sqoop2: Add test cases for the object name with special char

2015-11-11 Thread Sqoop QA bot (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2682?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001794#comment-15001794 ] Sqoop QA bot commented on SQOOP-2682: - Testing file [SQOOP-2682.001.patch|https://iss

Review Request 40216: SQOOP-2682: Sqoop2: Add test cases for the object name with special char

2015-11-11 Thread Colin Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40216/ --- Review request for Sqoop. Repository: sqoop-sqoop2 Description --- Add t

[jira] [Updated] (SQOOP-2682) Sqoop2: Add test cases for the object name with special char

2015-11-11 Thread Colin Ma (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Ma updated SQOOP-2682: Attachment: SQOOP-2682.001.patch > Sqoop2: Add test cases for the object name with special char > --

[jira] [Updated] (SQOOP-2682) Sqoop2: Add test cases for the object name with special char

2015-11-11 Thread Colin Ma (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Ma updated SQOOP-2682: Description: Add test cases for the object name with special char, eg, "." , "/", "?", "&" etc. (was: Add t

[jira] [Updated] (SQOOP-2682) Sqoop2: Add test cases for the object name with special char

2015-11-11 Thread Colin Ma (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2682?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Colin Ma updated SQOOP-2682: Summary: Sqoop2: Add test cases for the object name with special char (was: Sqoop2: Add test cases for the

Re: Review Request 40123: SQOOP-2675: Sqoop2: Remove the id from public interface for Job

2015-11-11 Thread Colin Ma
> On Nov. 10, 2015, 4:07 p.m., Jarek Cecho wrote: > > server/src/main/java/org/apache/sqoop/handler/HandlerUtils.java, lines 30-75 > > > > > > I like how this method is simplified suddenly :) > > > > One poin

Re: Missing Connectors in Sqoop2

2015-11-11 Thread Jonathan Seidman
Thanks Jarcec. I'll go ahead and prepare a patch. > On Nov 11, 2015, at 1:05 PM, Jarek Jarcec Cecho wrote: > > Good work Jonathan! > > I’m failing to see what is the purpose of the whole build section in most of > the pom files - perhaps we should just drop them all? I’m happy to review a >

[jira] [Created] (SQOOP-2682) Sqoop2: Add test cases for the connector name with special char

2015-11-11 Thread Colin Ma (JIRA)
Colin Ma created SQOOP-2682: --- Summary: Sqoop2: Add test cases for the connector name with special char Key: SQOOP-2682 URL: https://issues.apache.org/jira/browse/SQOOP-2682 Project: Sqoop Issue Ty

[jira] [Commented] (SQOOP-2677) Sqoop2: RESTiliency: Refactore the one line try-catch statement in InvalidRESTTest

2015-11-11 Thread Sqoop QA bot (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001306#comment-15001306 ] Sqoop QA bot commented on SQOOP-2677: - Testing file [SQOOP-2677.patch|https://issues.

Re: Missing Connectors in Sqoop2

2015-11-11 Thread Banmeet Singh
Thanks for the catch Jonathan. I too was suspecting an issue with the names but was not sure if it was intended to be that way for some reason. :) The change you suggested displays all the connectors; thanks again! Thanks and regards Banmeet Singh Graduate Student Department of Computer Science Ru

[jira] [Updated] (SQOOP-2677) Sqoop2: RESTiliency: Refactore the one line try-catch statement in InvalidRESTTest

2015-11-11 Thread Jarek Jarcec Cecho (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Jarcec Cecho updated SQOOP-2677: -- Fix Version/s: 1.99.7 > Sqoop2: RESTiliency: Refactore the one line try-catch statement

[jira] [Updated] (SQOOP-2677) Sqoop2: RESTiliency: Refactore the one line try-catch statement in InvalidRESTTest

2015-11-11 Thread Jarek Jarcec Cecho (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Jarcec Cecho updated SQOOP-2677: -- Attachment: SQOOP-2677.patch Very simple patch, so I'm skipping review board. > Sqoop2:

Re: Missing Connectors in Sqoop2

2015-11-11 Thread Jarek Jarcec Cecho
Good work Jonathan! I’m failing to see what is the purpose of the whole build section in most of the pom files - perhaps we should just drop them all? I’m happy to review a patch if you have patch available :) Jarcec > On Nov 11, 2015, at 12:29 PM, Jonathan Seidman > wrote: > > Hi Banmeet

[jira] [Updated] (SQOOP-2681) Sqoop2: Connectors failing to register

2015-11-11 Thread Jarek Jarcec Cecho (JIRA)
[ https://issues.apache.org/jira/browse/SQOOP-2681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jarek Jarcec Cecho updated SQOOP-2681: -- Priority: Blocker (was: Major) Let's call this a blocker. > Sqoop2: Connectors failing

Re: Missing Connectors in Sqoop2

2015-11-11 Thread Jonathan Seidman
Hi Banmeet – I'm also seeing this issue, so I opened SQOOP-2681. This appears to be caused by the connector jars (except for the Kafka connector) getting created with identical file names (sqoop.jar), which then get copied to the server/lib directory. I'm assuming the last file copied to the server

[jira] [Created] (SQOOP-2681) Sqoop2: Connectors failing to register

2015-11-11 Thread Jonathan Seidman (JIRA)
Jonathan Seidman created SQOOP-2681: --- Summary: Sqoop2: Connectors failing to register Key: SQOOP-2681 URL: https://issues.apache.org/jira/browse/SQOOP-2681 Project: Sqoop Issue Type: Bug

[GitHub] sqoop pull request: phoenix integration with 1.4.6

2015-11-11 Thread ravimagham
Github user ravimagham commented on the pull request: https://github.com/apache/sqoop/pull/10#issuecomment-155887939 Thanks Gabriel Reid. I will make the necessary corrections and have the patch updated to the ticket and open a RB. --- If your project is set up for it, you can reply

[GitHub] sqoop pull request: phoenix integration with 1.4.6

2015-11-11 Thread gabrielreid
Github user gabrielreid commented on the pull request: https://github.com/apache/sqoop/pull/10#issuecomment-155878871 Just taking a look through this -- having Phoenix support in Sqoop would be great! I know that this is an initial cut, but just a few remarks on the code: