Is there a user request to use this in an older version?
How easy is it to backport?
From the comments, it looks like it is needed for Geode artifacts published to
Maven? Is this true?
If there is no user request, and there is other way to include Tomcat session,
my view is to not backport, but I am not expert in this area, if there is
recommendation to backport, I am fine to. And if it has to be backported, it
should be on the version widely used and above(say 1.10 and above, again
depending on how easy to backport)...
-Anil.
On 1/11/21, 10:10 AM, "Sarah Abbey" <sab...@vmware.com> wrote:
Hey, Geode Devs!
Ben Ross and I are currently working on session state in Geode. In order
to include the Geode Tomcat session module in the Geode artifacts published to
Maven, we had to update the module so it publishes to Maven (as seen in this
PR<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5610&data=04%7C01%7Cagingade%40vmware.com%7Cdcf516a9acf44ebec0f408d8b65c3b81%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637459854565274941%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=RqiKTdtx7K8xCwHWqo3WlYCsSGVfM6PK%2FHNO8TQiFTo%3D&reserved=0>
and this
PR<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5762&data=04%7C01%7Cagingade%40vmware.com%7Cdcf516a9acf44ebec0f408d8b65c3b81%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C0%7C637459854565284938%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=%2BONGUDRAPVce88jzRYirRotMXnCWWkntzfr6jMZJGAI%3D&reserved=0>).
This change is only on the current develop branch. We are now wondering if
these changes should be backported to older versions of Geode.
What does everyone think? If it should be backported, to which versions
should it be backported?
Thank you,
Sarah
