Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2503
@manishgupta88 `SegmentFileName != null ` check is added only to
distinguish legacy or not, not for partition table. if the segment is new then
segmentfile will be present so that carbondata path can be taken relative to
the table path and generate blockid. I feel it is always better to take the
carbondata file path relative to the tablepath to avoid these types of checks
otherwise whenever we change the path we have to update the code.
---
