[BUG] [Star tree] Nodes with -1 as dimension value gets overriden with star node

[bharath-techie]

Describe the bug

Documents which has dimensions as '-1' , results in getting overriden with star nodes as star nodes are assigned '-1' when constructing star tree.

[-1,1]
[1,1]
[1,1]

Map<Long, InMemoryTreeNode> children = constructNonStarNodes(startDocId, endDocId, childDimensionId);
children.put((long) ALL, constructStarNode(startDocId, endDocId, childDimensionId));

ALL here is '-1'

Related component

Indexing:Performance

To Reproduce

Add documents with -1 as dimensions when creating star tree

Expected behavior

Documents with -1, null and any values in Long range should work with star tree

Additional Details

Plugins
Please list all plugins currently enabled.

Screenshots
If applicable, add screenshots to help explain your problem.

Host/Environment (please complete the following information):

• OS: [e.g. iOS]
• Version [e.g. 22]

Additional context
Add any other context about the problem here.

[mgodwan]

Lets get rid of

    public static final int ALL = -1;

in TreeNode. That would ensure we have a good rewrite.

Having a starChild separately should help

[bharath-techie]

Fixed as part of #14809