akashrn5 commented on a change in pull request #4129:
URL: https://github.com/apache/carbondata/pull/4129#discussion_r642884717
##########
File path:
core/src/main/java/org/apache/carbondata/core/scan/executor/util/RestructureUtil.java
##########
@@ -50,6 +52,10 @@
* Utility class for restructuring
*/
public class RestructureUtil {
+ // if table column is of complex type- this look up stores the column id of
the parent and their
+ // children. This helps to determine the existence of incoming query column
by matching based
+ // on id.
+ private static Map<String, String> existingTableColumnIDMap = new
HashMap<>();
Review comment:
its ok, instead of initializing every time, let the map contain other
column info also, no problem, you can keep one map only
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
