Anyone available to review/merge pull request for AVRO-1723 ?

2017-04-25 Thread Zoltan Farkas
let me know thank you —Z

[jira] [Commented] (AVRO-1957) TimeConversions do not implement getRecommendedSchema()

2017-04-25 Thread Suraj Acharya (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15983785#comment-15983785 ] Suraj Acharya commented on AVRO-1957: - Sorry that was due to a configuration error. I have fixed it

[jira] [Commented] (AVRO-1723) Add support for forward declarations in avro IDL

2017-04-25 Thread Zoltan Farkas (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1723?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15983499#comment-15983499 ] Zoltan Farkas commented on AVRO-1723: - Can somebody review the pull request? thank you > Add support

Re: Pre-Commit Build Jobs on Apache AVRO

2017-04-25 Thread Suraj Acharya
Sorry, I used a few terminology straight out of YETUS. * Modules are usually sub projects. In our case every language has been setup to be a module. Was a bit easier that way. * One of the main tests it runs is ./build.sh on the module(language) which had the change present. The determination of

[jira] [Commented] (AVRO-1957) TimeConversions do not implement getRecommendedSchema()

2017-04-25 Thread Sean Timm (JIRA)
[ https://issues.apache.org/jira/browse/AVRO-1957?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15983277#comment-15983277 ] Sean Timm commented on AVRO-1957: - [~hadoopqa] reported whitespace issues. I fixed three lines in the unit

RE: Pre-Commit Build Jobs on Apache AVRO

2017-04-25 Thread Gill, John
Suraj, Few questions for clarifications: - What do you mean by module - Java, C, Python or like an individual code file? - When you say, " ./build.sh test on the module you have made the change to" how does it determine this? - Once merged what will trigger the PreCommit Build?