Uploaded image for project: 'Phoenix'
  1. Phoenix
  2. PHOENIX-3610

Fix tableName used to get the index maintainers while creating HalfStoreFileReader for local index store

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 4.10.0
    • None
    • None

    Description

      Physical Tablename is used instead of phoenix table name . IndexHalfStoreFileReaderGenerator#preStoreFileReaderOpen

      TableName tableName = ctx.getEnvironment().getRegion().getTableDesc().getTableName();
      ..............
      try {
      
      
                      conn = 
      QueryUtil.getConnectionOnServer(ctx.getEnvironment().getConfiguration()).unwrap(
                                  PhoenixConnection.class);
                      PTable dataTable = PhoenixRuntime.getTableNoCache(conn, tableName.getNameAsString());
                      List<PTable> indexes = dataTable.getIndexes();
                      Map<ImmutableBytesWritable, IndexMaintainer> indexMaintainers =
                              new HashMap<ImmutableBytesWritable, IndexMaintainer>();
                      for (PTable index : indexes) {
                          if (index.getIndexType() == IndexType.LOCAL) {
                              IndexMaintainer indexMaintainer = index.getIndexMaintainer(dataTable, conn);
                              indexMaintainers.put(new ImmutableBytesWritable(MetaDataUtil
                                      .getViewIndexIdDataType().toBytes(index.getViewIndexId())),
                                  indexMaintainer);
                          }
                      }
                      if(indexMaintainers.isEmpty()) return reader;
                      byte[][] viewConstants = getViewConstants(dataTable);
                      return new IndexHalfStoreFileReader(fs, p, cacheConf, in, size, r, ctx
                              .getEnvironment().getConfiguration(), indexMaintainers, viewConstants,
                              childRegion, regionStartKeyInHFile, splitKey);
      

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ankit@apache.org Ankit Singhal
            ankit@apache.org Ankit Singhal
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment