Unify es_type as symbol or string
Summary
Ensure all usages of es_type
is the same type (either symbol or string).
Currently our es_type
classes provides string, yet many branching conditions checks for symbols.
Improvements
Avoids potential bugs when es_type
is used inside branching conditions.
I raise this because I met this type of issue and spent a few hours trying to dig the cause. (e.g. !22676 (bf31c677))
Risks
n/a
Involved components
Codes under ee/lib/elastic/**/*
Optional: Intended side effects
Optional: Missing test coverage
Edited by Mark Chao