Tagar commented on issue #6674: Fix NPE at io.druid.cli.CliInternalHadoopIndexer (fixes #6673) URL: https://github.com/apache/incubator-druid/pull/6674#issuecomment-442581984 Got it. Thanks @jihoonson So this check isn't possible as verify() at https://github.com/apache/incubator-druid/blob/d5633dd396a5cd16f98cb0e65d24ff5f8d6b33c3/indexing-hadoop/src/main/java/org/apache/druid/indexer/HadoopDruidIndexerConfig.java#L580 doesn't have a way of knowing if this spec is for `Cli Hadoop indexer` or for `hadoop index task` ? That NPE error I described at https://github.com/apache/incubator-druid/issues/6673 is very cryptic and only way to know that `metadataUpdateSpec` was missing is to look at the source code.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
