You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Veena Basavaraj <vb...@cloudera.com> on 2014/11/05 16:20:36 UTC

Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

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

Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/#review59981
-----------------------------------------------------------

Ship it!


Ship It!

- Jarek Cecho


On Nov. 5, 2014, 5:12 p.m., Veena Basavaraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27618/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2014, 5:12 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1674
>     https://issues.apache.org/jira/browse/SQOOP-1674
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   docs/src/site/sphinx/index.rst 1bea5c3 
> 
> Diff: https://reviews.apache.org/r/27618/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Veena Basavaraj
> 
>


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/
-----------------------------------------------------------

(Updated Nov. 5, 2014, 9:12 a.m.)


Review request for Sqoop.


Changes
-------

minor language fixes


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs (updated)
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/
-----------------------------------------------------------

(Updated Nov. 5, 2014, 9:06 a.m.)


Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs (updated)
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/
-----------------------------------------------------------

(Updated Nov. 5, 2014, 9:02 a.m.)


Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs (updated)
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/
-----------------------------------------------------------

(Updated Nov. 5, 2014, 8:56 a.m.)


Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs (updated)
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/
-----------------------------------------------------------

(Updated Nov. 5, 2014, 8:53 a.m.)


Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs (updated)
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/
-----------------------------------------------------------

(Updated Nov. 5, 2014, 8:41 a.m.)


Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs (updated)
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.

> On Nov. 5, 2014, 8:28 a.m., Jarek Cecho wrote:
> > docs/src/site/sphinx/index.rst, line 55
> > <https://reviews.apache.org/r/27618/diff/2/?file=750331#file750331line55>
> >
> >     It seems to me that the REST guide is more targetting the developpers then users, so perhaps we should keep it in the developer section?
> 
> Veena Basavaraj wrote:
>     The developers are the ones who will contribute code to connectors. May we can be explist about that.
>     
>     The users are ones who create the links and jobs via REST or some advanced users might just like the shell
> 
> Veena Basavaraj wrote:
>     When I update HUE, I would also add another 5 min demo link to the user guide to do the same things from HUE and rest apis

convinced now that the rest apis are means for UI developers to create a nice UI for sqoop internals as well. So it should be more of the developer guide.


- Veena


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


On Nov. 5, 2014, 8:41 a.m., Veena Basavaraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27618/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2014, 8:41 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1674
>     https://issues.apache.org/jira/browse/SQOOP-1674
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   docs/src/site/sphinx/index.rst 1bea5c3 
> 
> Diff: https://reviews.apache.org/r/27618/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Veena Basavaraj
> 
>


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.

> On Nov. 5, 2014, 8:28 a.m., Jarek Cecho wrote:
> > docs/src/site/sphinx/index.rst, line 55
> > <https://reviews.apache.org/r/27618/diff/2/?file=750331#file750331line55>
> >
> >     It seems to me that the REST guide is more targetting the developpers then users, so perhaps we should keep it in the developer section?
> 
> Veena Basavaraj wrote:
>     The developers are the ones who will contribute code to connectors. May we can be explist about that.
>     
>     The users are ones who create the links and jobs via REST or some advanced users might just like the shell

When I update HUE, I would also add another 5 min demo link to the user guide to do the same things from HUE and rest apis


- Veena


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


On Nov. 5, 2014, 7:59 a.m., Veena Basavaraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27618/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2014, 7:59 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1674
>     https://issues.apache.org/jira/browse/SQOOP-1674
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   docs/src/site/sphinx/index.rst 1bea5c3 
> 
> Diff: https://reviews.apache.org/r/27618/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Veena Basavaraj
> 
>


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Jarek Cecho <ja...@apache.org>.

> On Nov. 5, 2014, 4:28 p.m., Jarek Cecho wrote:
> > docs/src/site/sphinx/index.rst, line 55
> > <https://reviews.apache.org/r/27618/diff/2/?file=750331#file750331line55>
> >
> >     It seems to me that the REST guide is more targetting the developpers then users, so perhaps we should keep it in the developer section?
> 
> Veena Basavaraj wrote:
>     The developers are the ones who will contribute code to connectors. May we can be explist about that.
>     
>     The users are ones who create the links and jobs via REST or some advanced users might just like the shell
> 
> Veena Basavaraj wrote:
>     When I update HUE, I would also add another 5 min demo link to the user guide to do the same things from HUE and rest apis
> 
> Veena Basavaraj wrote:
>     convinced now that the rest apis are means for UI developers to create a nice UI for sqoop internals as well. So it should be more of the developer guide.

Yeah I see the confusion - there is multiple different categories of developpers. We have a "user based developper" who is someone that will create the HUE UI or integrate Sqoop into their application (Oozie would be great example). On the other hand we will have a connector developper who will create a connector with new data store. Even though that they are both 'developpers' they will care about different things.


- Jarek


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


On Nov. 5, 2014, 5:12 p.m., Veena Basavaraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27618/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2014, 5:12 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1674
>     https://issues.apache.org/jira/browse/SQOOP-1674
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   docs/src/site/sphinx/index.rst 1bea5c3 
> 
> Diff: https://reviews.apache.org/r/27618/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Veena Basavaraj
> 
>


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.

> On Nov. 5, 2014, 8:28 a.m., Jarek Cecho wrote:
> > docs/src/site/sphinx/index.rst, line 55
> > <https://reviews.apache.org/r/27618/diff/2/?file=750331#file750331line55>
> >
> >     It seems to me that the REST guide is more targetting the developpers then users, so perhaps we should keep it in the developer section?

The developers are the ones who will contribute code to connectors. May we can be explist about that.

The users are ones who create the links and jobs via REST or some advanced users might just like the shell


- Veena


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


On Nov. 5, 2014, 7:59 a.m., Veena Basavaraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27618/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2014, 7:59 a.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1674
>     https://issues.apache.org/jira/browse/SQOOP-1674
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   docs/src/site/sphinx/index.rst 1bea5c3 
> 
> Diff: https://reviews.apache.org/r/27618/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Veena Basavaraj
> 
>


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/#review59968
-----------------------------------------------------------



docs/src/site/sphinx/index.rst
<https://reviews.apache.org/r/27618/#comment101289>

    It seems to me that the REST guide is more targetting the developpers then users, so perhaps we should keep it in the developer section?


- Jarek Cecho


On Nov. 5, 2014, 3:59 p.m., Veena Basavaraj wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27618/
> -----------------------------------------------------------
> 
> (Updated Nov. 5, 2014, 3:59 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1674
>     https://issues.apache.org/jira/browse/SQOOP-1674
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> see jira
> 
> 
> Diffs
> -----
> 
>   docs/src/site/sphinx/index.rst 1bea5c3 
> 
> Diff: https://reviews.apache.org/r/27618/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Veena Basavaraj
> 
>


Re: Review Request 27618: SQOOP-1674:index.rst updates for Sqoop

Posted by Veena Basavaraj <vb...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/27618/
-----------------------------------------------------------

(Updated Nov. 5, 2014, 7:59 a.m.)


Review request for Sqoop.


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


Repository: sqoop-sqoop2


Description
-------

see jira


Diffs (updated)
-----

  docs/src/site/sphinx/index.rst 1bea5c3 

Diff: https://reviews.apache.org/r/27618/diff/


Testing
-------


Thanks,

Veena Basavaraj