Lamine
Lamine
Thank you @madrob for your time reviewing this. I have pushed a new commit and left some answers to your comments. Will add the ref guide as soon as this...
> I think we need to have a bigger discussion around backwards compatibility and how we achieve that. > > I would prefer to keep everything nice and neat, instead...
Hi @HoustonPutman, as discussed, removed the acl package and added the ref guide, plus other minor changes.
Thank you @madrob and @HoustonPutman for your review. Pushed the changes.
> Are we sure this fits as a Solr module? Since this is client-side solrj code, it could be in e.g. `solrj-aws-secret` instead? Actually, this is a server side code....
Can you please assign this to me after triage?
Hi @janhoy If you have some time, would you please take a look at this?
> I wish I knew more about this space, overall the changes make sense... Hi @epugh Any idea who can review this PR?
> I'm going to ping @janhoy on this ticket... If he isnt' able to review it, I can look some more early next week... > > Would you mind pinging...
Hi @msfroh I really like the idea. I'd like to work on this, can you please assign the issue to me?