Re: Review needed: ATLAS-2547

2018-04-13 Thread Nixon Rodrigues
Hi Pierre, The tried to apply your patch ATLAS-2478-2.patch, the build was successful with *-Pdist,berkeley-elasticsearch* profile, and Atlas UI loaded with all inbuild types, quick_script run fine and data loaded on UI. Found few issues. * Audits are not displaying on UI * Getting this

Re: Review needed: ATLAS-2547

2018-04-12 Thread Pierre Padovani
I could swear I had uploaded the diff to the review board. I updated the review with the diff, and added an additional sentence around hbase/solr. As near as I can tell, the change doesn't affect new installations, only existing that are upgrading. The only thing that will likely need to be done

Re: Review needed: ATLAS-2547

2018-04-12 Thread Madhan Neethiraj
Pierre, The review is in 'Discarded' state and no source file changes. I see a note on 'backwards incompatible' introduced in this patch. It is not clear what the impact will be on existing deployments that use HBase+Solr. Can you please add details? Thanks, Madhan On 4/12/18, 9:48 AM,

Re: Review needed: ATLAS-2547

2018-04-12 Thread Nixon Rodrigues
Pierre, I will review and validate it. Nixon On Thu, Apr 12, 2018, 10:17 PM Pierre Padovani < pierre.padov...@civitaslearning.com> wrote: > Hey all, > > I submitted a review to fix the Elasticsearch support several weeks ago, > and it has gotten no traction. > >