Hi Otis, I don’t mind doing any of that - but the problem is that such a change could impact backward compatibility - so we’d need to keep the stubs around even though the actual functionality might be elsewhere.
Thanks, Hari On Monday, November 11, 2013 at 3:54 PM, Otis Gospodnetic wrote: > Hi, > > Thanks for the info, everyone. > Yes, I noticed after my email that Blob* classes were in the process > of being moved. > Here is what I feel should really be done: > > * get rid of ....solr.morphline package and move the code to > ...morphpline package > * get rid of any Solr-specific code (I guess just in the tests > Wolfgang mentioned) > * rename the sink to MorphlineSink > > Thoughts? > > Re loadElasticSearch() - yes, I see Wolfgang saw I opened an issue for > that in CDK. > > Thanks, > Otis > -- > Performance Monitoring * Log Analytics * Search Analytics > Solr & Elasticsearch Support * http://sematext.com/ > > > On Mon, Nov 11, 2013 at 4:34 PM, Roshan Naik <[email protected] > (mailto:[email protected])> wrote: > > We should consider rename the Morphline Solr Sink to Morphline sink in the > > docs to avoid any possibility of misleading end users. > > > > -- > > CONFIDENTIALITY NOTICE > > NOTICE: This message is intended for the use of the individual or entity to > > which it is addressed and may contain information that is confidential, > > privileged and exempt from disclosure under applicable law. If the reader > > of this message is not the intended recipient, you are hereby notified that > > any printing, copying, dissemination, distribution, disclosure or > > forwarding of this communication is strictly prohibited. If you have > > received this communication in error, please contact the sender immediately > > and delete it from your system. Thank You. > > > > >
