Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-2802

The Java client tableExists should use GetTableSchema

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.0
    • Fix Version/s: 1.10.0
    • Component/s: None
    • Labels:
      None

      Description

      Currently the Java client `tableExists()` method uses a ListTables rpc. Instead it should use a GetTableSchema rpc. 

      Based on a comment in this review it could be especially important when authorization via Sentry is enabled:

      ...tableExists() is a bad idea if Sentry integration is enabled. That's because tableExists() results in a ListTables RPC, and ListTables is going to perform really poorly with a cold Sentry cache: it'll send an RPC to Sentry for every table in the catalog. Coupled with the loop here (for each table being backed up) and it's bad news all around...

        Attachments

          Activity

            People

            • Assignee:
              oclarms Xu Yao
              Reporter:
              granthenke Grant Henke
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: