> On 21 Jan 2016, at 09:15, Manoj Samel <manojsamelt...@gmail.com> wrote: > > Hello, > > Any update on this ? Should I file a Jira ?
always. It's the way to stop us forgetting things > > Thanks, > > On Tue, Jan 19, 2016 at 8:58 AM, Manoj Samel <manojsamelt...@gmail.com> > wrote: > >> Hi Jon, >> >> Did you get a chance to try to reproduce the issue with component >> containing a dash ? >> >> Thanks, >> >> Manoj >> >> On Thu, Jan 14, 2016 at 10:42 AM, Gour Saha <gs...@hortonworks.com> wrote: >> >>> Ok great. >>> >>> Steve, >>> This is interesting. I don’t see anywhere we validate the component name >>> against a dash. Am I missing something in the code :( >>> >>> -Gour >>> >>> On 1/14/16, 10:33 AM, "Manoj Samel" <manojsamelt...@gmail.com> wrote: >>> >>>> @Gour - yes, it does work when component name does not have a dash. It >>>> also >>>> works when component name has underscore(s) >>>> >>>> Thanks, >>>> >>>> Manoj >>>> >>>> On Thu, Jan 14, 2016 at 10:25 AM, Gour Saha <gs...@hortonworks.com> >>> wrote: >>>> >>>>> Ah, my bad. Nevertheless can I assume the app works fine when you do >>> not >>>>> include dash in component name? >>>>> >>>>> -Gour >>>>> >>>>> On 1/14/16, 9:42 AM, "Manoj Samel" <manojsamelt...@gmail.com> wrote: >>>>> >>>>>> Hi Gour, >>>>>> >>>>>> 1. The issue is with component name. The code you pointed is for >>>>> cluster >>>>>> name >>>>>> 2. As mentioned above, the component name does not seem to allow dash >>>>> "-", >>>>>> it does allow underscore. >>>>>> >>>>>> Thanks, >>>>>> >>>>>> Manoj >>>>>> >>>>>> On Thu, Jan 14, 2016 at 9:24 AM, Gour Saha <gs...@hortonworks.com> >>>>> wrote: >>>>>> >>>>>>> The current acceptable name pattern is [a-z][a-z0-9_-]* >>>>>>> (SliderUtils.java >>>>>>> - clusternamePattern). It includes the dash (-). >>>>>>> >>>>>>> While Jon experiments with your package, just wondering your app >>>>> works >>>>>>> fine when you get rid of the dash, right? >>>>>>> >>>>>>> -Gour >>>>>>> >>>>>>> On 1/14/16, 8:35 AM, "Jon Maron" <jma...@hortonworks.com> wrote: >>>>>>> >>>>>>>> Sorry - got pulled off on some other tasks. I¹ll try to take a >>> look >>>>>>> over >>>>>>>> the next day or so. >>>>>>>> >>>>>>>>> On Jan 14, 2016, at 11:31 AM, Manoj Samel >>>>> <manojsamelt...@gmail.com> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>> Hi, >>>>>>>>> >>>>>>>>> Any update on this ? Did you get a chance to try the memcache >>> with >>>>>>> the >>>>>>>>> component name I used above ? >>>>>>>>> >>>>>>>>> Thanks, >>>>>>>>> >>>>>>>>> Manoj >>>>>>>>> >>>>>>>>> On Tue, Jan 12, 2016 at 4:37 PM, Steve Loughran >>>>>>> <ste...@hortonworks.com >>>>>>>> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>>> >>>>>>>>>>> On 12 Jan 2016, at 12:14, Jon Maron <jma...@hortonworks.com> >>>>> wrote: >>>>>>>>>>> >>>>>>>>>>> OK. So was there a Œ-Œ or some other character in your >>>>> component >>>>>>>>>>> name? >>>>>>>>>> A Œ-Œ should work. The component names are currently expected >>> to >>>>>>>>>> follow a >>>>>>>>>> naming convention that allows for DNS compatible names, and >>>>> dashes >>>>>>> are >>>>>>>>>> included in that character set. The fact that the endpoint did >>>>> not >>>>>>>>>> appear >>>>>>>>>> may be related to some other issue. The AM logs may help here >>> as >>>>>>> well. >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> We should be checking component names early on, because it would >>>>>>> really >>>>>>>>>> break bits of the REST API too. >>>>>>>> >>>>>>> >>>>>>> >>>>> >>>>> >>> >>> >>