This is an automated email from the ASF dual-hosted git repository. ravipesala pushed a commit to branch branch-1.6 in repository https://gitbox.apache.org/repos/asf/carbondata.git
commit 7571b99ecb257956a05528a0d2910f66f4fd7daf Author: akashrn5 <[email protected]> AuthorDate: Thu Aug 15 17:56:57 2019 +0530 [CARBONDATA-3494]Fix NullPointerException in drop table and Correct the document formatting This closes #Problem: Fix the formatting of the document in index server md file. drop table is calling drop datamap command with force drop as true. Due to this table is removed from meta and physically. Then when processData is called for drop table, it tried to create carbonTable object by reading schema which causes NullPointerException. Solution: correct the formatting Skip ProcessData if carbonTable is null This closes #3359 --- .../org/apache/carbondata/core/datamap/DataMapStoreManager.java | 7 ++++++- docs/index-server.md | 6 ++---- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java b/core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java index ce0d6a6..f1f48fa 100644 --- a/core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java +++ b/core/src/main/java/org/apache/carbondata/core/datamap/DataMapStoreManager.java @@ -598,6 +598,11 @@ public final class DataMapStoreManager { */ public void deleteDataMap(AbsoluteTableIdentifier identifier, String dataMapName) { CarbonTable carbonTable = getCarbonTable(identifier); + if (carbonTable == null) { + // If carbon table is null then it means table is already deleted, therefore return without + // doing any further changes. + return; + } String tableUniqueName = identifier.getCarbonTableIdentifier().getTableUniqueName(); if (CarbonProperties.getInstance() .isDistributedPruningEnabled(identifier.getDatabaseName(), identifier.getTableName())) { @@ -613,7 +618,7 @@ public final class DataMapStoreManager { if (tableIndices != null) { int i = 0; for (TableDataMap tableDataMap : tableIndices) { - if (carbonTable != null && tableDataMap != null && dataMapName + if (tableDataMap != null && dataMapName .equalsIgnoreCase(tableDataMap.getDataMapSchema().getDataMapName())) { try { DataMapUtil diff --git a/docs/index-server.md b/docs/index-server.md index 5dd15c5..9253f2a 100644 --- a/docs/index-server.md +++ b/docs/index-server.md @@ -136,11 +136,9 @@ The Index Server is a long running service therefore the 'spark.yarn.keytab' and | Name | Default Value | Description | |:----------:|:-------------:|:------: | | carbon.enable.index.server | false | Enable the use of index server for pruning for the whole application. | -| carbon.index.server.ip | NA | Specify the IP/HOST on which the server is started. Better to - specify the private IP. | +| carbon.index.server.ip | NA | Specify the IP/HOST on which the server is started. Better to specify the private IP. | | carbon.index.server.port | NA | The port on which the index server is started. | -| carbon.disable.index.server.fallback | false | Whether to enable/disable fallback for index server -. Should be used for testing purposes only. Refer: [Fallback](#Fallback)| +| carbon.disable.index.server.fallback | false | Whether to enable/disable fallback for index server. Should be used for testing purposes only. Refer: [Fallback](#Fallback)| |carbon.index.server.max.jobname.length|NA|The max length of the job to show in the index server service UI. For bigger queries this may impact performance as the whole string would be sent from JDBCServer to IndexServer.|
