gerlowskija commented on pull request #565: URL: https://github.com/apache/solr/pull/565#issuecomment-1022613929
> Put the new ones inside, leave the old ones? It wouldn't be my choice in a vacuum: I can live with giving up on my long-term goal of reusing these payload classes, but introducing the inconsistency for different payloads just seems like kicking me while I'm down 😛 But I guess the inconsistency is only a problem if I'm slow getting a PR together to move the other `Payload` files. Let's go with this approach. Feel free to bug me in a few days if I don't ping you on a PR to move the rest of the payloads @madrob -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org