Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Merge Collection/Map using + Operator #247

Open
hantsy opened this issue Jul 26, 2024 · 3 comments
Open

Support Merge Collection/Map using + Operator #247

hantsy opened this issue Jul 26, 2024 · 3 comments

Comments

@hantsy
Copy link

hantsy commented Jul 26, 2024

To simplify the operators of concatenating Collections/Maps.

  • aList + aSet returns a new list
  • aMap+ anotherMap returns a new Map.
@markt-asf
Copy link
Contributor

This needs to be more precisely defined. I'd lean towards referencing an existing method (addAll(Collection) / putAll(Map)) rather than trying to define our own merge function. Cleaner to write the spec, easier for users to understand and easy to implement.

@markt-asf
Copy link
Contributor

I have an implementation of this in the current Tomcat 12 development branch. You can download SNAPSHOT build a standard distribution from:
https://repository.apache.org/content/groups/snapshots/org/apache/tomcat/tomcat/12.0.0-M1-SNAPSHOT/

Embedded and individual JARs are also available.

If you could try that and and confirm whether or not it works for you that would be great. I can then work on the changes required to the specification document and some TCK tests.

@markt-asf
Copy link
Contributor

It has been over a month without feedback. The operator works as expected for me so I am going to proceed with the spec updates and TCK tests for 6.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants