Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2245#discussion_r184937474
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -335,10 +335,12 @@ private CarbonTable buildCarbonTable() {
// When dimensions are default set to sort column,
// Inverted index will be supported by default for sort columns.
for (Field field : schema.getFields()) {
- if (field.getDataType() == DataTypes.STRING ||
- field.getDataType() == DataTypes.DATE ||
- field.getDataType() == DataTypes.TIMESTAMP) {
- sortColumnsList.add(field.getFieldName());
+ if (null != field) {
--- End diff --
Please add comment when field can be null
---
