Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
No Format
WIP VB

 

Some common practices that would make the reviewers happy when reviewing sqoop code

  • Do not send large patches ( A patch that is more than one page is frowned upon!). Break down the problem into as many granular sub-tasks  as possible. See SQOOP-1350 as one such example, there are other too of course. None of the RBs exceeded more than a page!
  • Do not send a RB without a link to the corresponding JIRA ticket
  • If there is a design wiki for the feature, please add that link to the RB so it is easy to get high level view of the do and don'ts. Kindly read the 
  • Follow the Sqoopesque religiously, no questions asked. If you find a new Sqoopesque, please leave a comment in that wiki