[
https://issues.apache.org/jira/browse/TRAFODION-2263?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15552375#comment-15552375
]
ASF GitHub Bot commented on TRAFODION-2263:
-------------------------------------------
Github user DaveBirdsall commented on a diff in the pull request:
https://github.com/apache/incubator-trafodion/pull/746#discussion_r82225420
--- Diff: core/sql/comexe/LateBindInfo.cpp ---
@@ -1319,37 +1049,102 @@ Int16 AnsiOrNskName::quoteNSKExtName()
return 0;
}
-void ResolvedNameListPre1800::translateFromNewVersion(ResolvedNameList
*newrnl)
+///////////////////////////////////////////////////////////////////
+// class TrafSimilarityTableInfo
+///////////////////////////////////////////////////////////////////
+TrafSimilarityTableInfo::TrafSimilarityTableInfo(char * tableName,
+ NABoolean isHive,
+ char * hdfsRootDir,
+ Int64 modTS, Int32
numPartnLevels,
+ Queue * hdfsDirsToCheck,
+ char * hdfsHostName,
+ Int32 hdfsPort)
+ : NAVersionedObject(-1),
+ tableName_(tableName),
+ hdfsRootDir_(hdfsRootDir),
+ modTS_(modTS), numPartnLevels_(numPartnLevels),
+ hdfsDirsToCheck_(hdfsDirsToCheck),
+ hdfsHostName_(hdfsHostName), hdfsPort_(hdfsPort)
{
- this->numEntries()= newrnl->numEntries();
- this->resetFlags();
- for (Int32 i = 0; i < (Int32) (newrnl->numEntries());i++)
- {
- strcpy(this->getResolvedName(i).resolvedGuardianName(),
- newrnl->getResolvedName(i).resolvedGuardianName());
-
-
this->getResolvedName(i).setIgnoreTS((short)newrnl->getResolvedName(i).ignoreTS());
-
this->getResolvedName(i).setResolvedAnsiName(newrnl->getResolvedName(i).resolvedAnsiName());
-
this->getResolvedName(i).setValidateNumIndexes((short)newrnl->getResolvedName(i).validateNumIndexes());
-
this->getResolvedName(i).setNumIndexes(newrnl->getResolvedName(i).numIndexes());
- }
+ if (isHive)
+ setIsHive(TRUE);
}
-void ResolvedNameList::translateFromOldVersion(ResolvedNameListPre1800
*oldrnl)
+TrafSimilarityTableInfo::TrafSimilarityTableInfo()
+ : NAVersionedObject(-1),
+ tableName_(NULL),
+ hdfsRootDir_(NULL),
+ modTS_(-1), numPartnLevels_(-1),
+ hdfsDirsToCheck_(NULL),
+ hdfsHostName_(NULL), hdfsPort_(NULL)
--- End diff --
Need a flags_ inintializer here too.
> Move Similarity check for hive tables to master executor root operator
> ----------------------------------------------------------------------
>
> Key: TRAFODION-2263
> URL: https://issues.apache.org/jira/browse/TRAFODION-2263
> Project: Apache Trafodion
> Issue Type: Improvement
> Reporter: Anoop Sharma
> Assignee: Anoop Sharma
> Priority: Minor
>
> Query plans for hive tables include location of data directories/files
> and the plan depends on it.
> If data changes in a hive file/dir after a query has been compiled and
> before it is executed, then that change is detected at runtime by
> comparing compile time and runtime dir timestamps.
> A mismatch causes that query to recompile using AQR and a new
> plan is generated with latest dir location.
> This detection is currently being done at leaf operators (scan, insert) when
> that operator is executed.
> This jira is to move that check (called Similarity Check) to the root
> operator before starting query execution. That way plan consistency
> checks will be done before running the query and will not depend
> on the actual operator being executed. It will also result in checks not
> being done at various processes and operators like an ESP process.
> All checks will be done at one place in master executor and code will not
> be spread across multiple operators as it is now.
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
