elastic / elasticsearch

Free and Open, Distributed, RESTful Search Engine
https://www.elastic.co/products/elasticsearch
Other
69.41k stars 24.56k forks source link

Deprecate Unnecessary Painless Whitelisted Methods #50295

Open stu-elastic opened 4 years ago

stu-elastic commented 4 years ago

There are a few items in the painless whitelist which don't seem to fit:

Observable

This class represents an observable object, or "data" in the model-view paradigm. It can be subclassed to represent an object that the application wants to have observed.

An observable object can have one or more observers. An observer may be any object that implements interface Observer. After an observable instance changes, an application calling the Observable's notifyObservers method causes all of its observers to be notified of the change by a call to their update method.

class java.util.Observable {
()
void addObserver(Observer)
int countObservers()
void deleteObserver(Observer)
void deleteObservers()
boolean hasChanged()
void notifyObservers()
void notifyObservers(Object)
}

EventListenerProxy

An abstract wrapper class for an EventListener class which associates a set of additional parameters with the listener. Subclasses must provide the storage and accessor methods for the additional arguments or parameters.

class java.util.EventListenerProxy {
EventListener getListener()
}

EventObject

All Events are constructed with a reference to the object, the "source", that is logically deemed to be the object upon which the Event in question initially occurred upon.

class java.util.EventObject {
(Object)
Object getSource()
}
elasticmachine commented 4 years ago

Pinging @elastic/es-core-infra (:Core/Infra/Scripting)