Awesome! thank you ... I'll post a patch later today after I've addressed the issues Gour pointed out.
On Wed, May 27, 2015 at 9:18 AM, Gour Saha <[email protected]> wrote: > Yes, please file a JIRA and submit as a patch. I have already tested the > package on CentOS6. Everything looks good, except that I am unable to see > ³zkhost² in servers export. I see ³host_port² only. > > Couple more points: > - Is this package platform independent? If not, do you plan to submit a > separate patch for Windows? If yes, would be great if it could be > mentioned in the README. > - Please ensure Apache license headers are added to all files including > README.md (refer to files under > https://github.com/apache/incubator-slider/blob/develop/app-packages/hbase > as example). I think the json files can be skipped. > - Remove the LICENSE file from the top-level dir > > -Gour > > On 5/27/15, 7:50 AM, "Ted Yu" <[email protected]> wrote: > >>Please open a JIRA. >> >>Thanks >> >>On Wed, May 27, 2015 at 6:24 AM, Timothy Potter <[email protected]> >>wrote: >> >>> Hi, >>> >>> I'm a committer on the Lucene/Solr project and have developed a basic >>> solution for deploying Solr to YARN using Slider, see: >>> https://github.com/LucidWorks/solr-slider >>> >>> I'm reaching out to see if there is interest in including Solr as one >>> of the packages that ships with Slider, as you do today with Storm, >>> memcached, HBase, etc ... >>> >>> If so, I can open a JIRA and prepare a patch based on my github >>> project, but wanted to gauge interest before going down that path. >>> >>> Cheers, >>> Timothy Potter >>> >
