We can now remove the deprecated methods GetLatestVersion and GetLatestSubVersion
authorrgrosso <rgrosso@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 21 Feb 2013 14:14:44 +0000 (14:14 +0000)
committerrgrosso <rgrosso@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 21 Feb 2013 14:14:44 +0000 (14:14 +0000)
from AliCDBStorage and derived classes, following revisions 61012 and 61018.
The issue was discussed in the savannah ticket #82052, which can now be closed.

STEER/CDB/AliCDBDump.cxx
STEER/CDB/AliCDBDump.h
STEER/CDB/AliCDBGrid.cxx
STEER/CDB/AliCDBGrid.h
STEER/CDB/AliCDBLocal.cxx
STEER/CDB/AliCDBLocal.h
STEER/CDB/AliCDBStorage.h

index 9f002a6..c292db2 100644 (file)
@@ -767,47 +767,6 @@ Bool_t AliCDBDump::IdToFilename(const AliCDBId& /*id*/, TString& /*filename*/) c
         return kFALSE;
 }
 
-//_____________________________________________________________________________
-Int_t AliCDBDump::GetLatestVersion(const char* path, Int_t run){
-// get last version found in the database valid for run and path
-
-       AliCDBPath aCDBPath(path);
-       if(!aCDBPath.IsValid() || aCDBPath.IsWildcard()) {
-               AliError(Form("Invalid path in request: %s", path));
-               return -1;
-       }
-
-       AliCDBId query(path, run, run, -1, -1);
-       AliCDBId *dataId = GetId(query);
-
-       if(!dataId) return -1;
-       Int_t version = dataId->GetVersion();
-
-       delete dataId;
-       return version;
-}
-
-//_____________________________________________________________________________
-Int_t AliCDBDump::GetLatestSubVersion(const char* path, Int_t run, Int_t version){
-// get last version found in the database valid for run and path
-
-       AliCDBPath aCDBPath(path);
-       if(!aCDBPath.IsValid() || aCDBPath.IsWildcard()) {
-               AliError(Form("Invalid path in request: %s", path));
-               return -1;
-       }
-
-       AliCDBId query(path, run, run, version, -1);
-       AliCDBId *dataId = GetId(query);
-
-       if(!dataId) return -1;
-
-       Int_t subVersion = dataId->GetSubVersion();
-
-       delete dataId;
-       return subVersion;
-
-}
 
 //_____________________________________________________________________________
 void AliCDBDump::SetRetry(Int_t /* nretry */, Int_t /* initsec */) {
index a339f57..f8500d9 100644 (file)
@@ -25,8 +25,6 @@ public:
        virtual Bool_t IsReadOnly() const {return fReadOnly;};
        virtual Bool_t HasSubVersion() const {return kFALSE;};
        virtual Bool_t Contains(const char* path) const;
-       virtual Int_t GetLatestVersion(const char* path, Int_t run);
-       virtual Int_t GetLatestSubVersion(const char* path, Int_t run, Int_t version=-1);
        virtual Bool_t IdToFilename(const AliCDBId& id, TString& filename) const;
        virtual void SetRetry(Int_t /* nretry */, Int_t /* initsec */);
 
index 3a85300..9d1885d 100644 (file)
@@ -1184,92 +1184,6 @@ void AliCDBGrid::MakeQueryFilter(Int_t firstRun, Int_t lastRun,
 
 }
 
-//_____________________________________________________________________________
-Int_t AliCDBGrid::GetLatestVersion(const char* path, Int_t run){
-// get last version found in the database valid for run and path
-
-       AliCDBPath aCDBPath(path);
-       if(!aCDBPath.IsValid() || aCDBPath.IsWildcard()) {
-               AliError(Form("Invalid path in request: %s", path));
-               return -1;
-       }
-       AliCDBId query(path, run, run, -1, -1);
-
-       AliCDBId* dataId = 0;
-       // look for file matching query requests (path, runRange, version)
-       if(run == fRun && fPathFilter.Comprises(aCDBPath) && fVersion < 0){
-               // look into list of valid files previously loaded with AliCDBStorage::FillValidFileIds()
-               AliDebug(2, Form("List of files valid for run %d and for path %s was loaded. Looking there!",
-                                       run, path));
-               dataId = GetId(fValidFileIds, query);
-               if (!dataId) return -1;
-               Int_t version = dataId->GetVersion();
-               delete dataId;
-               return version;
-
-       }
-       // List of files valid for reqested run was not loaded. Looking directly into CDB
-       AliDebug(2, Form("List of files valid for run %d and for path %s was not loaded. Looking directly into CDB!",
-                               run, path));
-
-       TObjArray validFileIds;
-       validFileIds.SetOwner(1);
-
-       TString filter;
-       MakeQueryFilter(run, run, 0, filter);
-
-       TString pattern = ".root";
-
-       TString folderCopy(Form("%s%s/Run",fDBFolder.Data(),path));
-
-       AliDebug(2,Form("** fDBFolder = %s, pattern = %s, filter = %s",folderCopy.Data(), pattern.Data(), filter.Data()));
-       TGridResult *res = gGrid->Query(folderCopy, pattern, filter, "-y -m");
-
-       AliCDBId validFileId;
-       if (res->GetEntries()>1){
-               AliWarning("Number of found entries >1, even if option -y was used");
-               for(int i=0; i<res->GetEntries(); i++){
-                       TString filename = res->GetKey(i, "lfn");
-                       if(filename == "") continue;
-                       if(FilenameToId(filename, validFileId))
-                               validFileIds.AddLast(validFileId.Clone());
-               }
-               dataId = GetId(validFileIds, query);
-               if (!dataId) return -1;
-               
-               Int_t version = dataId->GetVersion();
-               delete dataId;
-               return version;
-       }
-       else if (res->GetEntries()==1){
-               TString filename = res->GetKey(0, "lfn");
-               if(filename == "") {
-                       AliError("The only entry found has filename empty");
-                       return -1;
-               }
-               if(FilenameToId(filename, validFileId)) return validFileId.GetVersion();
-               else{
-                       AliError("Impossible to get FileId from filename");
-                       return -1;
-               }
-       }
-       else {
-               AliError("No entries found");
-               return -1;
-       }
-
-       delete res;
-
-
-}
-
-//_____________________________________________________________________________
-Int_t AliCDBGrid::GetLatestSubVersion(const char* /*path*/, Int_t /*run*/, Int_t /*version*/){
-// get last subversion found in the database valid for run and path
-       AliError("Objects in GRID storage have no sub version!");
-       return -1;
-}
-
 
 /////////////////////////////////////////////////////////////////////////////////////////////////
 //                                                                                             //
index e1f684e..9e9de8b 100644 (file)
@@ -23,8 +23,6 @@ public:
        virtual Bool_t IsReadOnly() const {return kFALSE;}
        virtual Bool_t HasSubVersion() const {return kFALSE;}
        virtual Bool_t Contains(const char* path) const;
-       virtual Int_t  GetLatestVersion(const char* path, Int_t run);
-       virtual Int_t  GetLatestSubVersion(const char* path, Int_t run, Int_t version);
        virtual Bool_t IdToFilename(const AliCDBId& id, TString& filename) const;
        virtual void SetRetry(Int_t nretry, Int_t initsec);
        virtual void SetMirrorSEs(const char* mirrors) {fMirrorSEs=mirrors;}
index 8ffddd2..ab4f8e6 100644 (file)
@@ -905,49 +905,6 @@ void AliCDBLocal::QueryValidFiles()
 
 }
 
-//_____________________________________________________________________________
-Int_t AliCDBLocal::GetLatestVersion(const char* path, Int_t run){
-// get last version found in the database valid for run and path
-
-       AliCDBPath aCDBPath(path);
-       if(!aCDBPath.IsValid() || aCDBPath.IsWildcard()) {
-               AliError(Form("Invalid path in request: %s", path));
-               return -1;
-       }
-
-       AliCDBId query(path, run, run, -1, -1);
-       AliCDBId* dataId = GetId(query);
-
-       if(!dataId) return -1;
-
-       Int_t version = dataId->GetVersion();
-       delete dataId;
-
-       return version;
-
-}
-
-//_____________________________________________________________________________
-Int_t AliCDBLocal::GetLatestSubVersion(const char* path, Int_t run, Int_t version){
-// get last version found in the database valid for run and path
-
-       AliCDBPath aCDBPath(path);
-       if(!aCDBPath.IsValid() || aCDBPath.IsWildcard()) {
-               AliError(Form("Invalid path in request: %s", path));
-               return -1;
-       }
-
-       AliCDBId query(path, run, run, version, -1);
-       AliCDBId *dataId = GetId(query);
-
-       if(!dataId) return -1;
-
-       Int_t subVersion = dataId->GetSubVersion();
-
-       delete dataId;
-       return subVersion;
-
-}
 
 /////////////////////////////////////////////////////////////////////////////////////////////////
 //                                                                                             //
index fc206ef..22f5e06 100644 (file)
@@ -22,8 +22,6 @@ public:
        virtual Bool_t IsReadOnly() const {return kFALSE;};
        virtual Bool_t HasSubVersion() const {return kTRUE;};
        virtual Bool_t Contains(const char* path) const;
-       virtual Int_t  GetLatestVersion(const char* path, Int_t run);
-       virtual Int_t  GetLatestSubVersion(const char* path, Int_t run, Int_t version=-1);
        virtual Bool_t IdToFilename(const AliCDBId& id, TString& filename) const;
        virtual void SetRetry(Int_t /* nretry */, Int_t /* initsec */);
 
index 620a825..d61ebd8 100644 (file)
@@ -100,9 +100,6 @@ public:
                        Int_t version=-1, AliCDBMetaData *mdFilter=0);
        void PrintQueryCDB();
        TObjArray* GetQueryCDBList() {return &fValidFileIds;}
-
-       virtual Int_t GetLatestVersion(const char* path, Int_t run)=0;
-       virtual Int_t GetLatestSubVersion(const char* path, Int_t run, Int_t version=-1)=0;
        virtual void SetRetry(Int_t nretry, Int_t initsec) = 0;
 
 protected: