Mailing-List: contact dev-help@drill.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@drill.apache.org
From: arina-ielchiieva <git@git.apache.org>
To: dev@drill.apache.org
Reply-To: dev@drill.apache.org
References: <git-pr-877-drill@git.apache.org>
In-Reply-To: <git-pr-877-drill@git.apache.org>
Subject: [GitHub] drill pull request #877: DRILL-5660: Drill 1.10 queries fail due to Parquet ...
Content-Type: text/plain
Message-Id: <20170721072057.2433CED30B@git1-us-west.apache.org>
Date: Fri, 21 Jul 2017 07:20:57 +0000 (UTC)
archived-at: Fri, 21 Jul 2017 07:20:59 -0000

Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/877#discussion_r128697396
  
    --- Diff: exec/java-exec/src/test/java/org/apache/drill/BaseTestQuery.java ---
    @@ -639,14 +644,18 @@ protected static void copyDirectoryIntoTempSpace(String resourcesDir, String des
        * @param srcFileOnClassPath the source path of metadata cache file, which should be replaced
        * @param destFolderInTmp  the parent folder name of the metadata cache file
        * @param metaFileName the name of metadata cache file depending on the type of the metadata
    +   * @param customStringReplacement custom string to replace the "CUSTOM_REPLACED" target string in metadata file
        * @throws IOException if a create or write errors occur
        */
    -  protected static void copyMetaDataCacheToTempReplacingInternalPaths(String srcFileOnClassPath, String destFolderInTmp,
    -      String metaFileName) throws IOException {
    +  protected static void copyMetaDataCacheToTempWithReplacements(String srcFileOnClassPath,
    +      String destFolderInTmp, String metaFileName, String customStringReplacement) throws IOException {
         String metadataFileContents = getFile(srcFileOnClassPath);
         Path rootMeta = new Path(dfsTestTmpSchemaLocation, destFolderInTmp);
         Path newMetaCache = new Path(rootMeta, metaFileName);
         FSDataOutputStream outSteam = fs.create(newMetaCache);
    +    if (customStringReplacement!=null) {
    --- End diff --
    
    `if (customStringReplacement!=null) {` -> `if (customStringReplacement != null) {`


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---